[Bug bootstrap/53021] New: [4.8 Regression] bootstrap failure on Linux/ia32

[Bug bootstrap/53021] New: [4.8 Regression] bootstrap failure on Linux/ia32

[hjl.tools at gmail dot com]

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

             Bug #: 53021
           Summary: [4.8 Regression] bootstrap failure on Linux/ia32
    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
                CC: rsandifo@gcc.gnu.org
            Target: i686-pc-linux-gnu


On Linux/ia32, revision 186540:

http://gcc.gnu.org/ml/gcc-cvs/2012-04/msg00491.html

caused:

/export/build/gnu/gcc-32bit/build-i686-linux/./gcc/xgcc
-B/export/build/gnu/gcc-32bit/build-i686-linux/./gcc/
-B/usr/gcc-4.8.0/i686-linux/bin/ -B/usr/gcc-4.8.0/i686-linux/lib/ -isystem
/usr/gcc-4.8.0/i686-linux/include -isystem
/usr/gcc-4.8.0/i686-linux/sys-include    -O2 -g -O2  -O2 -g -DIN_GCC   -W -Wall
-Wwrite-strings -Wcast-qual -Wstrict-prototypes -Wmissing-prototypes
-Wold-style-definition  -isystem ./include   -fpic -g -DIN_LIBGCC2
-fbuilding-libgcc -fno-stack-protector   -fpic -I. -I. -I../.././gcc
-I/export/gnu/import/git/gcc/libgcc -I/export/gnu/import/git/gcc/libgcc/.
-I/export/gnu/import/git/gcc/libgcc/../gcc
-I/export/gnu/import/git/gcc/libgcc/../include
-I/export/gnu/import/git/gcc/libgcc/config/libbid -DENABLE_DECIMAL_BID_FORMAT
-DHAVE_CC_TLS  -DUSE_TLS -o unwind-dw2-fde-dip_s.o -MT unwind-dw2-fde-dip_s.o
-MD -MP -MF unwind-dw2-fde-dip_s.dep -DSHARED -fexceptions -c
/export/gnu/import/git/gcc/libgcc/unwind-dw2-fde-dip.c
/export/gnu/import/git/gcc/libgcc/unwind-dw2-fde-dip.c:461:1: internal compiler
error: Segmentation fault
 }
 ^
Please submit a full bug report,
with preprocessed source if appropriate.
See <http://gcc.gnu.org/bugs.html> for instructions.
make[2]: *** [unwind-dw2-fde-dip_s.o] Error 1
make[2]: Leaving directory
`/export/build/gnu/gcc-32bit/build-i686-linux/i686-linux/libgcc'
make[1]: *** [all-target-libgcc] Error 2

[Bug bootstrap/53021] [4.8 Regression] bootstrap failure on Linux/ia32

[hjl.tools at gmail dot com]

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

--- Comment #1 from H.J. Lu <hjl.tools at gmail dot com> 2012-04-17 19:57:44 UTC ---
unique_base_value calls gen_rtx_ADDRESS which overrides
dwarf_file_data created by lookup_filename in dwarf2out.c:

 /* Check to see if the file name that was searched on the previous
     call matches this file name.  If so, return the index.  */
  if (file_table_last_lookup
      && (file_name == file_table_last_lookup->filename
          || filename_cmp (file_table_last_lookup->filename, file_name) == 0))
    return file_table_last_lookup;

  /* Didn't match the previous lookup, search the table.  */
  slot = htab_find_slot_with_hash (file_table, file_name,
                                   htab_hash_string (file_name), INSERT);
  if (*slot)                             
    return (struct dwarf_file_data *) *slot;

  created = ggc_alloc_dwarf_file_data ();
  created->filename = file_name;
  created->emitted_number = 0;
  *slot = created;
  return created;

However, return of htab_find_slot_with_hash wasn't marked with
GTY(()) and file_table_last_lookup was never set.

[Bug bootstrap/53021] [4.8 Regression] bootstrap failure on Linux/ia32

[rsandifo at gcc dot gnu.org]

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

--- Comment #2 from rsandifo at gcc dot gnu.org <rsandifo at gcc dot gnu.org> 2012-04-17 20:09:11 UTC ---
Author: rsandifo
Date: Tue Apr 17 20:09:01 2012
New Revision: 186549

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=186549
Log:
gcc/
    PR bootstrap/53021
    * rtl.c (rtx_code_size): Handle ADDRESS.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/rtl.c

[Bug bootstrap/53021] [4.8 Regression] bootstrap failure on Linux/ia32

[ebotcazou at gcc dot gnu.org]

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

Eric Botcazou <ebotcazou at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2012-04-18
                 CC|                            |ebotcazou at gcc dot
                   |                            |gnu.org
   Target Milestone|---                         |4.8.0
     Ever Confirmed|0                           |1
           Severity|normal                      |critical

--- Comment #3 from Eric Botcazou <ebotcazou at gcc dot gnu.org> 2012-04-18 10:24:45 UTC ---
Another one (with RTL checking enabled):

../../../src/libgcc/libgcc2.c: In function '__muldi3':
../../../src/libgcc/libgcc2.c:559:1: internal compiler error: RTL check:
expected elt 0 type 'e' or 'u', have 'w' (rtx address) in record_store, at
dse.c:1506
 }
 ^
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:mailto:report@adacore.com> for instructions.
make[3]: *** [_muldi3.o] Error 1
make[3]: Leaving directory 

Please bootstrap with --enable-checking=yes,rtl when you're doing RTL surgery,
this is cheap enough now.

[Bug bootstrap/53021] [4.8 Regression] bootstrap failure on Linux/ia32

[jakub at gcc dot gnu.org]

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

--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> 2012-04-20 12:20:07 UTC ---
Author: jakub
Date: Fri Apr 20 12:19:51 2012
New Revision: 186623

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=186623
Log:
    PR bootstrap/53021
    * alias.h (UNIQUE_BASE_VALUE_SP, UNIQUE_BASE_VALUE_ARGP,
    UNIQUE_BASE_VALUE_FP, UNIQUE_BASE_VALUE_HFP): Define.
    * alias.c (init_alias_targets): Use UNIQUE_BASE_VALUE_*
    macros instead of constants.
    * dse.c (record_store): Check for SP ADDRESS by comparing
    XWINT to UNIQUE_BASE_VALUE_SP instead of expecting
    XEXP to be stack_pointer_rtx.

Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/alias.c
    trunk/gcc/alias.h
    trunk/gcc/dse.c

[Bug bootstrap/53021] [4.8 Regression] bootstrap failure on Linux/ia32

[rsandifo at gcc dot gnu.org]

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

rsandifo@gcc.gnu.org <rsandifo at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
         AssignedTo|unassigned at gcc dot       |rsandifo at gcc dot gnu.org
                   |gnu.org                     |

--- Comment #5 from rsandifo at gcc dot gnu.org <rsandifo at gcc dot gnu.org> 2012-04-21 10:23:10 UTC ---
Looks like there's another case in store-motion.c.  I'm making
another pass at cleaning up this whole mess -- sorry again for
making such a horlicks of it all.

[Bug bootstrap/53021] [4.8 Regression] bootstrap failure on Linux/ia32

[rsandifo at gcc dot gnu.org]

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

rsandifo@gcc.gnu.org <rsandifo at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED

--- Comment #7 from rsandifo at gcc dot gnu.org <rsandifo at gcc dot gnu.org> 2012-04-21 18:57:01 UTC ---
Patch applied.

[Bug bootstrap/53021] [4.8 Regression] bootstrap failure on Linux/ia32

[rsandifo at gcc dot gnu.org]

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

--- Comment #6 from rsandifo at gcc dot gnu.org <rsandifo at gcc dot gnu.org> 2012-04-21 18:55:22 UTC ---
Author: rsandifo
Date: Sat Apr 21 18:55:18 2012
New Revision: 186657

URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=186657
Log:
gcc/
    PR bootstrap/53021
    * rtl.def (ADDRESS): Use "i" rather than "w".
    * rtl.h (find_base_term): Delete.
    (may_be_sp_based_p): Declare.
    * rtl.c (rtx_code_size): Remove ADDRESS special case.
    * alias.h (UNIQUE_BASE_VALUE_SP, UNIQUE_BASE_VALUE_ARGP)
    (UNIQUE_BASE_VALUE_FP, UNIQUE_BASE_VALUE_HFP): Move to...
    * alias.c: ...here.
    (find_base_term): Make static.
    (may_be_sp_based_p): New function.
    * dse.c (record_store): Use it.
    * store-motion.c (store_killed_in_insn): Likewise.


Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/alias.c
    trunk/gcc/alias.h
    trunk/gcc/dse.c
    trunk/gcc/rtl.c
    trunk/gcc/rtl.def
    trunk/gcc/rtl.h
    trunk/gcc/store-motion.c