http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60155
--- Comment #12 from Jeffrey A. Law law at redhat dot com ---
Dave,
The updated patch looks good. Please post it to gcc-patches for archival
purposes with a note that I approved it in BZ before installing the patch.
Thanks,
jeff
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60155
--- Comment #13 from John David Anglin danglin at gcc dot gnu.org ---
Author: danglin
Date: Fri Apr 4 22:25:51 2014
New Revision: 209134
URL: http://gcc.gnu.org/viewcvs?rev=209134root=gccview=rev
Log:
PR rtl-optimization/60155
* gcse.c
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60155
John David Anglin danglin at gcc dot gnu.org changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60155
--- Comment #11 from dave.anglin at bell dot net ---
On 3-Mar-14, at 8:01 PM, danglin at gcc dot gnu.org wrote:
Bah, doesn't fix bug:
Attached new patch which seems to fix bug. Testing.
--
John David Anglindave.ang...@bell.net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60155
--- Comment #10 from John David Anglin danglin at gcc dot gnu.org ---
Bah, doesn't fix bug:
dave@mx3210:~/gnu/gcc/objdir$ gcc/xgcc -Bgcc/ -g -O2 -Wall -Wpointer-arith
-Wuninitialized -Wsign-compare -Wformat-security -Wsizeof-pointer-memaccess
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60155
--- Comment #9 from dave.anglin at bell dot net ---
Something like this?
--
John David Anglindave.ang...@bell.net
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60155
--- Comment #7 from dave.anglin at bell dot net ---
On 25-Feb-14, at 1:36 AM, law at redhat dot com wrote:
It really feels like this is papering over the real problem, namely
that
get_pressure_class_and_nregs simply doesn't handle things
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60155
--- Comment #8 from Jeffrey A. Law law at redhat dot com ---
Rather than special case TRAP_IF, I think we just need to walk through the
pattern. If we find no assignments or 1 assignment, then we assert.
note_stores, or walking similar to
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60155
Jeffrey A. Law law at redhat dot com changed:
What|Removed |Added
CC||law at redhat dot
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60155
John David Anglin danglin at gcc dot gnu.org changed:
What|Removed |Added
CC||mikulas at
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60155
--- Comment #4 from John David Anglin danglin at gcc dot gnu.org ---
Created attachment 32140
-- http://gcc.gnu.org/bugzilla/attachment.cgi?id=32140action=edit
Patch
The attach patch fixes the ICE but maybe we should always check
if the INSN
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60155
--- Comment #1 from John David Anglin danglin at gcc dot gnu.org ---
With 4.6 and 4.7 compilers, this appears as:
gcc-4.6 -g -O2 -Wall -Wpointer-arith -Wuninitialized -Wsign-compare
-Wformat-security -Wno-pointer-sign -Wno-unused-result
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60155
--- Comment #2 from John David Anglin danglin at gcc dot gnu.org ---
Breakpoint 1, get_pressure_class_and_nregs (insn=0xfab51d98, nregs=0xfaf028c0)
at ../../gcc/gcc/gcse.c:3459
3459 gcc_assert (set != NULL_RTX);
(gdb) p debug_rtx (insn)
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60155
--- Comment #3 from John David Anglin danglin at gcc dot gnu.org ---
Function compiles without -ftrapv.
14 matches
Mail list logo