[Bug rtl-optimization/59340] [4.9 Regression] LRA enable-checking bootstrap failure since r205136

2013-12-02 Thread krebbel at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59340

Andreas Krebbel krebbel at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Andreas Krebbel krebbel at gcc dot gnu.org ---
Bootstrap works again with r205541. Thanks!


[Bug rtl-optimization/59340] [4.9 Regression] LRA enable-checking bootstrap failure since r205136

2013-11-29 Thread rguenth at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59340

Richard Biener rguenth at gcc dot gnu.org changed:

   What|Removed |Added

   Keywords||ice-checking
 Target||s390x-*-*
   Target Milestone|--- |4.9.0
Summary|LRA enable-checking |[4.9 Regression] LRA
   |bootstrap failure since |enable-checking bootstrap
   |r205136 |failure since r205136


[Bug rtl-optimization/59340] [4.9 Regression] LRA enable-checking bootstrap failure since r205136

2013-11-29 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59340

Jakub Jelinek jakub at gcc dot gnu.org changed:

   What|Removed |Added

 CC||jakub at gcc dot gnu.org

--- Comment #1 from Jakub Jelinek jakub at gcc dot gnu.org ---
Perhaps we want a different flag for the recog call than lra_in_progress?


[Bug rtl-optimization/59340] [4.9 Regression] LRA enable-checking bootstrap failure since r205136

2013-11-29 Thread vmakarov at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59340

Vladimir Makarov vmakarov at gcc dot gnu.org changed:

   What|Removed |Added

 CC||vmakarov at gcc dot gnu.org

--- Comment #2 from Vladimir Makarov vmakarov at gcc dot gnu.org ---
(In reply to Jakub Jelinek from comment #1)
 Perhaps we want a different flag for the recog call than lra_in_progress?

Or don't use it all, it is too heavy.  It is better to use something light.  I
added this code on early stages of LRA development because it is hard to
predict what RTL could be.  Most important check before was one on addresses
and it is gone.  Insn_invalid_p check is just a paranoid check.  If something
goes wrong, the current LRA catches it later in any case.

I hope to fix it today.


[Bug rtl-optimization/59340] [4.9 Regression] LRA enable-checking bootstrap failure since r205136

2013-11-29 Thread vmakarov at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59340

--- Comment #3 from Vladimir Makarov vmakarov at gcc dot gnu.org ---
Author: vmakarov
Date: Fri Nov 29 20:08:38 2013
New Revision: 205541

URL: http://gcc.gnu.org/viewcvs?rev=205541root=gccview=rev
Log:
2013-11-29  Vladimir Makarov  vmaka...@redhat.com

PR rtl-optimization/59340
* lra.c (check_rtl): Use recog_memoized instead of insn_invalid_p.

Revert
2013-11-20  Robert Suchanek  robert.sucha...@imgtec.com

* lra.c (lra): Set lra_in_progress before check_rtl call.
* recog.c (insn_invalid_p): Add !lra_in_progress to prevent
adding clobber regs when LRA is running.


Modified:
trunk/gcc/ChangeLog
trunk/gcc/lra.c
trunk/gcc/recog.c


[Bug rtl-optimization/59340] [4.9 Regression] LRA enable-checking bootstrap failure since r205136

2013-11-29 Thread vmakarov at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59340

--- Comment #4 from Vladimir Makarov vmakarov at gcc dot gnu.org ---
Andreas, could you check that the bootstrap is fixed.  At least, the test is
compiled by the cross-compiler.