[Bug middle-end/82556] [7/8 Regression] internal compiler error in curr_insn_transform, at lra-constraints.c:4307

2017-11-22 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82556

Jakub Jelinek  changed:

   What|Removed |Added

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

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

[Bug middle-end/82556] [7/8 Regression] internal compiler error in curr_insn_transform, at lra-constraints.c:4307

2017-10-18 Thread vmakarov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82556

--- Comment #8 from Vladimir Makarov  ---
Author: vmakarov
Date: Wed Oct 18 16:47:38 2017
New Revision: 253863

URL: https://gcc.gnu.org/viewcvs?rev=253863&root=gcc&view=rev
Log:
2017-10-18  Vladimir Makarov  

PR middle-end/82556
* lra-constraints.c (curr_insn_transform): Use non-input operand
instead of output one for matched reload.

2017-10-18  Vladimir Makarov  

PR middle-end/82556
* gcc.target/i386/pr82556.c: New.


Added:
branches/gcc-7-branch/gcc/testsuite/gcc.target/i386/pr82556.c
Modified:
branches/gcc-7-branch/gcc/ChangeLog
branches/gcc-7-branch/gcc/lra-constraints.c
branches/gcc-7-branch/gcc/testsuite/ChangeLog

[Bug middle-end/82556] [7/8 Regression] internal compiler error in curr_insn_transform, at lra-constraints.c:4307

2017-10-18 Thread vmakarov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82556

--- Comment #7 from Vladimir Makarov  ---
Author: vmakarov
Date: Wed Oct 18 16:44:27 2017
New Revision: 253862

URL: https://gcc.gnu.org/viewcvs?rev=253862&root=gcc&view=rev
Log:
2017-10-18  Vladimir Makarov  

PR middle-end/82556
* lra-constraints.c (curr_insn_transform): Use non-input operand
instead of output one for matched reload.

2017-10-18  Vladimir Makarov  

PR middle-end/82556
* gcc.target/i386/pr82556.c: New.


Added:
trunk/gcc/testsuite/gcc.target/i386/pr82556.c
Modified:
trunk/gcc/ChangeLog
trunk/gcc/lra-constraints.c
trunk/gcc/testsuite/ChangeLog

[Bug middle-end/82556] [7/8 Regression] internal compiler error in curr_insn_transform, at lra-constraints.c:4307

2017-10-17 Thread vmakarov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82556

--- Comment #6 from Vladimir Makarov  ---
(In reply to Jakub Jelinek from comment #5)
> Indeed started with my r253710.  If there is something wrong on the
> constraints on the patterns, please let me know.

The constraints are ok.  I'll probably have and submit the LRA patch tomorrow.

[Bug middle-end/82556] [7/8 Regression] internal compiler error in curr_insn_transform, at lra-constraints.c:4307

2017-10-17 Thread jakub at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82556

--- Comment #5 from Jakub Jelinek  ---
Indeed started with my r253710.  If there is something wrong on the constraints
on the patterns, please let me know.

[Bug middle-end/82556] [7/8 Regression] internal compiler error in curr_insn_transform, at lra-constraints.c:4307

2017-10-17 Thread vmakarov at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82556

--- Comment #4 from Vladimir Makarov  ---
(In reply to Richard Biener from comment #3)
> Confirmed on the branch (for the unreduced testcase).  GCC 7.2 works.
> 
> Probably Jakubs pattern changes.

It is an unusual pattern for LRA.  I guess it should be fixed in LRA.  I'll
investigate the problem and try to find a fix in LRA.

[Bug middle-end/82556] [7/8 Regression] internal compiler error in curr_insn_transform, at lra-constraints.c:4307

2017-10-17 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=82556

Richard Biener  changed:

   What|Removed |Added

   Priority|P3  |P1
 CC||jakub at gcc dot gnu.org
  Known to work||7.2.0
   Target Milestone|8.0 |7.3
Summary|[8 Regression] internal |[7/8 Regression] internal
   |compiler error in   |compiler error in
   |curr_insn_transform, at |curr_insn_transform, at
   |lra-constraints.c:4307  |lra-constraints.c:4307
  Known to fail||7.2.1, 8.0

--- Comment #3 from Richard Biener  ---
Confirmed on the branch (for the unreduced testcase).  GCC 7.2 works.

Probably Jakubs pattern changes.