http://gcc.gnu.org/bugzilla/show_bug.cgi?id=55030



--- Comment #13 from Eric Botcazou <ebotcazou at gcc dot gnu.org> 2013-02-18 
10:48:18 UTC ---

> I've lost track. What was "the original patch", what do you mean by "the

> blockage patch" (that has been installed) and I'm pretty sure there were

> several follow-up patches, so I can't say I'm confident about reverting

> something from just this subset.  (To wit: if it's something that causes

> volatile asms to again be treated different from (other) blockages, then 
> that's

> wrong, as a volatile asm is the default blockage.)



The original patch was the patch you reverted in comment #1 and the fix for the

original issue.  As far as I can see, you didn't put it back after installing

the blockage patch, which is the patch in comment #5.  The dse.c and cselib.c

hunks were not necessary and not in the submission, but I asked you to add them

for the sake of consistency; then Jakub pointed out that they pessimize

volatile asms.

Reply via email to