This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug middle-end/55030] [4.8 Regression]: gcc.c-torture/execute/builtins/memcpy-chk.c execution, -Os (et al)
- From: "ebotcazou at gcc dot gnu.org" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: Mon, 18 Feb 2013 10:48:18 +0000
- Subject: [Bug middle-end/55030] [4.8 Regression]: gcc.c-torture/execute/builtins/memcpy-chk.c execution, -Os (et al)
- Auto-submitted: auto-generated
- References: <bug-55030-4@http.gcc.gnu.org/bugzilla/>
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.