AVR C Runtime Library - Patches: patch #7909, Adding __volatile__ to __asm__...
You are not allowed to post comments on this tracker with your current authentication level.
patch #7909: Adding __volatile__ to __asm__ within pgmspace header
Submitter: | Alex Crawford <crawford> | ||
Submitted: | Mon 31 Dec 2012 05:54:17 AM UTC | ||
Category: | None | Priority: | 5 - Normal |
Status: | Done | Privacy: | Public |
Assigned to: | joerg_wunsch | Open/Closed: | Closed |
Thu 02 May 2013 03:56:12 PM UTC, comment #2: |
Joerg Wunsch <joerg_wunsch> |
Fri 04 Jan 2013 01:15:30 PM UTC, comment #1: FYI, I filed this as bug #38021
|
Georg-Johann Lay <gjlayde> |
Mon 31 Dec 2012 05:54:17 AM UTC, original submission:
I noticed that gcc was optimizing out calls to pgm_read_word_far() that were within a while loop. Adding the volatile directive fixes the underlying problem. I've added the directive to each of the LPM and ELPM calls (20 in all).
|
Alex Crawford <crawford> |
Depends on the following items: None found
Items that depend on this one: None found
There are 0 votes so far. Votes easily highlight which items people would like to see resolved in priority, independently of the priority of the item set by tracker managers.
Follow 4 latest changes.
Date | Changed by | Updated Field | Previous Value | => | Replaced by |
---|---|---|---|---|---|
2013-05-02 | joerg_wunsch | Status | None | Done | |
Assigned to | None | joerg_wunsch | |||
Open/Closed | Open | Closed | |||
2012-12-31 | crawford | Attached File | - | Added volatile.patch, #27168 |
Powered by Savane 3.13-02a9.
Corresponding source code
Applied in SVN r2381.
I'll keep bug #38021 open anyway, but the patch at least
appears to be better than the existing implementation, so
I think it's useful as a short-term solution.