Bug 49563 - Expansion forgets to set MEM_VOLATILE_P for TARGET_MEM_REF
Summary: Expansion forgets to set MEM_VOLATILE_P for TARGET_MEM_REF
Status: UNCONFIRMED
Alias: None
Product: gcc
Classification: Unclassified
Component: middle-end (show other bugs)
Version: 4.7.0
: P3 normal
Target Milestone: ---
Assignee: Not yet assigned to anyone
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-06-28 09:05 UTC by Duncan Sands
Modified: 2011-06-28 09:18 UTC (History)
1 user (show)

See Also:
Host:
Target:
Build:
Known to work:
Known to fail:
Last reconfirmed:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Duncan Sands 2011-06-28 09:05:53 UTC
In expr.c, this code

        if (TREE_THIS_VOLATILE (exp))
          MEM_VOLATILE_P (temp) = 1;

sets MEM_VOLATILE_P for a volatile MEM_REF.  But this is not done for a volatile
TARGET_MEM_REF.  This doesn't matter right now because tree-ssa-loop-ivopts.c
bails out if it sees a volatile memory reference, so volatile TARGET_MEM_REFs
don't (yet) occur in the wild.
Comment 1 Richard Biener 2011-06-28 09:18:30 UTC
It looks like set_mem_attributes () will set it via a similar check.