This is the mail archive of the gcc-bugs@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[Bug middle-end/55030] [4.8 Regression]: gcc.c-torture/execute/builtins/memcpy-chk.c execution, -Os (et al)


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.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]