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

2012-06-28 Thread jakub at gcc dot gnu.org
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53555

Jakub Jelinek  changed:

   What|Removed |Added

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

--- Comment #7 from Jakub Jelinek  2012-06-28 
14:24:35 UTC ---
Fixed.


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

2012-06-04 Thread 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  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=gcc&view=rev&rev=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

2012-06-02 Thread hjl.tools at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53555

H.J. Lu  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  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

2012-06-02 Thread hjl.tools at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53555

--- Comment #4 from H.J. Lu  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

2012-06-02 Thread izamyatin at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53555

Igor Zamyatin  changed:

   What|Removed |Added

 CC||izamyatin at gmail dot com

--- Comment #3 from Igor Zamyatin  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

2012-06-01 Thread hjl.tools at gmail dot com
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53555

H.J. Lu  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  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