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

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

[hjl.tools at gmail dot com]

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

             Bug #: 53555
           Summary: [4.8 Regression] Bootstrap failure
    Classification: Unclassified
           Product: gcc
           Version: 4.8.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: bootstrap
        AssignedTo: unassigned@gcc.gnu.org
        ReportedBy: hjl.tools@gmail.com


On Linux/ia32, revision 188107 failed to bootstrap when
configured with

../src-trunk/configure \
       --with-arch=core2 --with-cpu=atom --prefix=/usr/local
--enable-clocale=gnu --with-system-zlib --enable-shared --with-demangler-in-ld
i686-linux --with-fpmath=sse --enable-languages=c,c++,fortran,java,lto,objc

Comparing stages 2 and 3
warning: gcc/cc1obj-checksum.o differs
warning: gcc/cc1plus-checksum.o differs
warning: gcc/cc1-checksum.o differs
Bootstrap comparison failure!
gcc/gimple-fold.o differs
gcc/jvspec.o differs
make[5]: *** [compare] Error 1

Revision 188104 is OK.

[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

[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 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=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

[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.