[Bug target/79242] [7 Regression] ICE in simplify_subreg, at simplify-rtx.c:6029

[law at gcc dot gnu.org]

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

--- Comment #13 from Jeffrey A. Law  ---
Author: law
Date: Sat Feb 17 00:25:15 2018
New Revision: 257779

URL: https://gcc.gnu.org/viewcvs?rev=257779=gcc=rev
Log:
2018-02-16  Jozef Lawrynowicz 

PR target/79242
* machmode.def: Define a complex mode for PARTIAL_INT.
* genmodes.c (complex_class): Return MODE_COMPLEX_INT for
MODE_PARTIAL_INT.
* doc/rtl.texi: Document CSPImode.
* config/msp430/msp430.c (msp430_hard_regno_nregs): Add CPSImode
handling.
(msp430_hard_regno_nregs_with_padding): Likewise.

PR target/79242
gcc.target/msp430/pr79242.c: New test.

Added:
branches/gcc-7-branch/gcc/testsuite/gcc.target/msp430/pr79242.c
Modified:
branches/gcc-7-branch/gcc/ChangeLog
branches/gcc-7-branch/gcc/config/msp430/msp430.c
branches/gcc-7-branch/gcc/doc/rtl.texi
branches/gcc-7-branch/gcc/genmodes.c
branches/gcc-7-branch/gcc/machmode.def
branches/gcc-7-branch/gcc/testsuite/ChangeLog

[Bug target/79242] [7 Regression] ICE in simplify_subreg, at simplify-rtx.c:6029

[law at redhat dot com]

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

Jeffrey A. Law  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #12 from Jeffrey A. Law  ---
Fixed on gcc-7 branch as well.

[Bug target/79242] [7 Regression] ICE in simplify_subreg, at simplify-rtx.c:6029

[jozefl.gcc at gmail dot com]

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

--- Comment #11 from Jozef Lawrynowicz  ---
(In reply to Eric Gallager from comment #10)
> (In reply to Jeffrey A. Law from comment #9)
> > Author: law
> > Date: Wed Feb 14 07:21:11 2018
> > New Revision: 257653
> > 
> > URL: https://gcc.gnu.org/viewcvs?rev=257653=gcc=rev
> > Log:
> > 2018-02-14  Jozef Lawrynowicz 
> > 
> > PR target/79242
> > * machmode.def: Define a complex mode for PARTIAL_INT.
> > * genmodes.c (complex_class): Return MODE_COMPLEX_INT for
> > MODE_PARTIAL_INT.
> > * doc/rtl.texi: Document CSPImode.
> > * config/msp430/msp430.c (msp430_hard_regno_nregs): Add CPSImode
> > handling.
> > (msp430_hard_regno_nregs_with_padding): Likewise.
> > 
> > PR target/79242
> > * gcc.target/msp430/pr79242.c: New test.
> > 
> > Added:
> > trunk/gcc/testsuite/gcc.target/msp430/pr79242.c
> > Modified:
> > trunk/gcc/ChangeLog
> > trunk/gcc/config/msp430/msp430.c
> > trunk/gcc/doc/rtl.texi
> > trunk/gcc/genmodes.c
> > trunk/gcc/machmode.def
> > trunk/gcc/testsuite/ChangeLog
> 
> Did this fix it?

The patch is currently only on trunk, but after back-porting it to
gcc-7-branch, GCC now builds successfully for the msp430-elf target.
The original testcase also compiles successfully with the resulting toolchain.

Trunk still doesn't build for msp430-elf though, due to Bug 84406.

[Bug target/79242] [7 Regression] ICE in simplify_subreg, at simplify-rtx.c:6029

[egallager at gcc dot gnu.org]

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

Eric Gallager  changed:

   What|Removed |Added

   Keywords||patch
 CC||egallager at gcc dot gnu.org

--- Comment #10 from Eric Gallager  ---
(In reply to Jeffrey A. Law from comment #9)
> Author: law
> Date: Wed Feb 14 07:21:11 2018
> New Revision: 257653
> 
> URL: https://gcc.gnu.org/viewcvs?rev=257653=gcc=rev
> Log:
> 2018-02-14  Jozef Lawrynowicz 
> 
>   PR target/79242
>   * machmode.def: Define a complex mode for PARTIAL_INT.
>   * genmodes.c (complex_class): Return MODE_COMPLEX_INT for
>   MODE_PARTIAL_INT.
>   * doc/rtl.texi: Document CSPImode.
>   * config/msp430/msp430.c (msp430_hard_regno_nregs): Add CPSImode
>   handling.
>   (msp430_hard_regno_nregs_with_padding): Likewise.
> 
>   PR target/79242
>   * gcc.target/msp430/pr79242.c: New test.
> 
> Added:
> trunk/gcc/testsuite/gcc.target/msp430/pr79242.c
> Modified:
> trunk/gcc/ChangeLog
> trunk/gcc/config/msp430/msp430.c
> trunk/gcc/doc/rtl.texi
> trunk/gcc/genmodes.c
> trunk/gcc/machmode.def
> trunk/gcc/testsuite/ChangeLog

Did this fix it?