[Bug middle-end/85414] [8 Regression] ICE: in ix86_expand_prologue, at config/i386/i386.c:13810 with -Og -fgcse

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85414
Bug 85414 depends on bug 90095, which changed state.

Bug 90095 Summary: [9 Regression] wrong code with -Os -fno-tree-bit-ccp
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=90095

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

[Bug middle-end/85414] [8 Regression] ICE: in ix86_expand_prologue, at config/i386/i386.c:13810 with -Og -fgcse

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85414

--- Comment #6 from Jakub Jelinek  ---
Author: jakub
Date: Wed Apr 25 13:10:01 2018
New Revision: 259649

URL: https://gcc.gnu.org/viewcvs?rev=259649=gcc=rev
Log:
PR middle-end/85414
* simplify-rtx.c (simplify_unary_operation_1) : Pass SUBREG_REG (op) rather than op to
gen_lowpart_no_emit.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/expr.c
trunk/gcc/simplify-rtx.c

[Bug middle-end/85414] [8 Regression] ICE: in ix86_expand_prologue, at config/i386/i386.c:13810 with -Og -fgcse

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85414

Jakub Jelinek  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Jakub Jelinek  ---
Fixed.

[Bug middle-end/85414] [8 Regression] ICE: in ix86_expand_prologue, at config/i386/i386.c:13810 with -Og -fgcse

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85414

--- Comment #4 from Jakub Jelinek  ---
Author: jakub
Date: Tue Apr 17 09:16:48 2018
New Revision: 259433

URL: https://gcc.gnu.org/viewcvs?rev=259433=gcc=rev
Log:
PR middle-end/85414
* rtlhooks.c (gen_lowpart_if_possible): Don't call gen_lowpart_SUBREG
on a SUBREG.

* gcc.dg/pr85414.c: New test.

Added:
trunk/gcc/testsuite/gcc.dg/pr85414.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/rtlhooks.c
trunk/gcc/testsuite/ChangeLog

[Bug middle-end/85414] [8 Regression] ICE: in ix86_expand_prologue, at config/i386/i386.c:13810 with -Og -fgcse

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=85414

Jakub Jelinek  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|unassigned at gcc dot gnu.org  |jakub at gcc dot gnu.org

--- Comment #3 from Jakub Jelinek  ---
Created attachment 43946
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=43946=edit
gcc8-pr85414.patch

Untested fix.  We really should never generate subreg of subreg, period.