[Bug bootstrap/53555] [4.8 Regression] Bootstrap failure

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53555

Jakub Jelinek jakub at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 CC||jakub at gcc dot gnu.org
 Resolution||FIXED

--- Comment #7 from Jakub Jelinek jakub at gcc dot gnu.org 2012-06-28 
14:24:35 UTC ---
Fixed.

[Bug bootstrap/53555] [4.8 Regression] Bootstrap failure

[hjl at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53555

--- Comment #6 from hjl at gcc dot gnu.org hjl at gcc dot gnu.org 2012-06-04 
23:31:57 UTC ---
Author: hjl
Date: Mon Jun  4 23:31:54 2012
New Revision: 188212

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=188212
Log:
Skip debug insns in ix86_sched_reorder

PR bootstrap/53555
* config/i386/i386.c (ix86_sched_reorder) Skip debug insns.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/i386/i386.c

[Bug bootstrap/53555] [4.8 Regression] Bootstrap failure

[izamyatin at gmail dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53555

Igor Zamyatin izamyatin at gmail dot com changed:

   What|Removed |Added

 CC||izamyatin at gmail dot com

--- Comment #3 from Igor Zamyatin izamyatin at gmail dot com 2012-06-02 
11:17:41 UTC ---
Hmm, it seems it's not quite correct to check ix86_tune, proper way is to use
ix86_arch here. Following change fixes he bootstrap

diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index c6c2c5e..a272411 100644
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -23881,7 +23881,7 @@ ix86_sched_reorder(FILE *dump, int sched_verbose, rtx
*ready, int *pn_ready,
   issue_rate = ix86_issue_rate();

   /* Do reodering for Atom only.  */
-  if (ix86_tune != PROCESSOR_ATOM)
+  if (ix86_arch != PROCESSOR_ATOM)
 return issue_rate;
   /* Nothing to do if ready list contains only 1 instruction.  */
   if (n_ready = 1)

[Bug bootstrap/53555] [4.8 Regression] Bootstrap failure

[hjl.tools at gmail dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53555

--- Comment #4 from H.J. Lu hjl.tools at gmail dot com 2012-06-02 12:07:59 
UTC ---
(In reply to comment #3)
 Hmm, it seems it's not quite correct to check ix86_tune, proper way is to use
 ix86_arch here. Following change fixes he bootstrap
 
 diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
 index c6c2c5e..a272411 100644
 --- a/gcc/config/i386/i386.c
 +++ b/gcc/config/i386/i386.c
 @@ -23881,7 +23881,7 @@ ix86_sched_reorder(FILE *dump, int sched_verbose, rtx
 *ready, int *pn_ready,
issue_rate = ix86_issue_rate();
 
/* Do reodering for Atom only.  */
 -  if (ix86_tune != PROCESSOR_ATOM)
 +  if (ix86_arch != PROCESSOR_ATOM)
  return issue_rate;
/* Nothing to do if ready list contains only 1 instruction.  */
if (n_ready = 1)

This is the wrong fix. Bootstrap will fail if GCC is configured
with --with-arch=atom.

[Bug bootstrap/53555] [4.8 Regression] Bootstrap failure

[hjl.tools at gmail dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53555

H.J. Lu hjl.tools at gmail dot com changed:

   What|Removed |Added

URL||http://gcc.gnu.org/ml/gcc-p
   ||atches/2012-06/msg00115.htm
   ||l
 CC||ubizjak at gmail dot com

--- Comment #5 from H.J. Lu hjl.tools at gmail dot com 2012-06-02 13:29:13 
UTC ---
A patch is posted at

http://gcc.gnu.org/ml/gcc-patches/2012-06/msg00115.html

[Bug bootstrap/53555] [4.8 Regression] Bootstrap failure

[hjl.tools at gmail dot com]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53555

H.J. Lu hjl.tools at gmail dot com changed:

   What|Removed |Added

 CC||areg.melikadamyan at gmail
   ||dot com, kirill.yukhin at
   ||intel dot com
   Target Milestone|--- |4.8.0

--- Comment #1 from H.J. Lu hjl.tools at gmail dot com 2012-06-01 13:44:28 
UTC ---
It may be caused by revision 188107:

http://gcc.gnu.org/ml/gcc-cvs/2012-06/msg00017.html