[Bug target/12624] New: Gcc failed to bootstrap on ia64

[Bug target/12624] New: Gcc failed to bootstrap on ia64

[hjl at lucon dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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

           Summary: Gcc failed to bootstrap on ia64
           Product: gcc
           Version: 3.4
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: hjl at lucon dot org
                CC: gcc-bugs at gcc dot gnu dot org
 GCC build triplet: ia64-unknown-linux-gnu
  GCC host triplet: ia64-unknown-linux-gnu
GCC target triplet: ia64-unknown-linux-gnu

As of Wed Oct 15 05:53:46 UTC 2003, I got many

In file included from /net/gnu/export/gnu/src/gcc/gcc/libjava/include/jvm.h:23,
                 from /net/gnu/export/gnu/src/gcc/gcc/libjava/jni.cc:18:
./include/java-threads.h:228: internal compiler error: Segmentation fault
Please submit a full bug report,
with preprocessed source if appropriate.
See <URL:http://gcc.gnu.org/bugs.html> for instructions.
make[5]: *** [jni.lo] Error 1

It seems related to

http://gcc.gnu.org/ml/gcc/2003-09/msg00311.html
http://gcc.gnu.org/ml/gcc/2003-09/msg00416.html

[Bug target/12624] Gcc failed to bootstrap on ia64

[hjl at lucon dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


hjl at lucon dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |hubicka at gcc dot gnu dot
                   |dot org                     |org
             Status|UNCONFIRMED                 |ASSIGNED

[Bug target/12624] Gcc failed to bootstrap on ia64

[wilson at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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



------- Additional Comments From wilson at gcc dot gnu dot org  2003-10-23 06:01 -------
Here is a minimal testcase.

register int foo __asm__ ("r13");

If you compile this with g++ at -O2 you get an ICE.  This is trivial to
reproduce with a cross compiler to ia64-linux.

The path to here is
  cp_finish_decl
  make_rtl_for_nonlocal_decl
  rest_of_decl_compilation
  cgraph_varpool_finalize_decl
  cgraph_varpool_mark_needed_node
  notice_global_symbol
and it dies in the last one because it assumes DECL_RTL contains a SYMBOL_REF.

It seems simple enough to change notice_global_symbol to test for a SYMBOL_REF
instead of assuming it.  A global register variable doesn't seem to be of any
use here.

The same testcase doesn't fail with the C compiler though.  This is because the
C front end sets DECL_DEFER_OUTPUT, but the C++ front end does not.  This causes
us to not call cgraph_varpool_finalize_decl when compiled as C code.

[Bug c++/12624] [3.4 Regression] [unit-at-a-time] ICE in notice_global_symbol

[pinskia at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
          Component|target                      |c++
  GCC build triplet|ia64-unknown-linux-gnu      |
   GCC host triplet|ia64-unknown-linux-gnu      |
 GCC target triplet|ia64-unknown-linux-gnu      |
           Keywords|                            |build, ice-on-valid-code
            Summary|Gcc failed to bootstrap on  |[3.4 Regression] [unit-at-a-
                   |ia64                        |time] ICE in
                   |                            |notice_global_symbol
   Target Milestone|---                         |3.4


------- Additional Comments From pinskia at gcc dot gnu dot org  2003-10-23 06:14 -------
I can reproduce this with the simplifed testcase on powerpc-apple-darwin so this is generic bug 
not effecting only ia64.
Here is the backtrace I have:

#0  0x001e877c in notice_global_symbol (decl=0x41831d20) at /Volumes/UFS_Partition/pinskia/
src/fsf/gcc-clean/src/gcc/varasm.c:1067
#1  0x003145c4 in cgraph_varpool_mark_needed_node (node=0x41833018) at /Volumes/
UFS_Partition/pinskia/src/fsf/gcc-clean/src/gcc/cgraph.c:476
#2  0x002fce10 in rest_of_decl_compilation (decl=0x41831d20, asmspec=0x41015a98 "r13", 
top_level=1, at_end=0) at /Volumes/UFS_Partition/pinskia/src/fsf/gcc-clean/src/gcc/toplev.c:
1950
#3  0x00021428 in cp_finish_decl (decl=0x41831d20, init=0x0, asmspec_tree=0x41831d20, 
flags=0) at /Volumes/UFS_Partition/pinskia/src/fsf/gcc-clean/src/gcc/cp/decl.c:4577
#4  0x0005912c in cp_parser_init_declarator (parser=0x440000, decl_specifiers=0x450000, 
prefix_attributes=0x0, function_definition_allowed_p=4521984, member_p=false, 
declares_class_or_enum=1099111712, function_definition_p=0x41015a98) at /Volumes/
UFS_Partition/pinskia/src/fsf/gcc-clean/src/gcc/cp/parser.c:9492
#5  0x00059454 in cp_parser_simple_declaration (parser=0x41015a98, 
function_definition_allowed_p=1099111712) at /Volumes/UFS_Partition/pinskia/src/fsf/gcc-
clean/src/gcc/cp/parser.c:6156
#6  0x00051b34 in cp_parser_block_declaration (parser=0x41831af0, statement_p=false) at /
Volumes/UFS_Partition/pinskia/src/fsf/gcc-clean/src/gcc/cp/parser.c:6072
#7  0x0005a7cc in cp_parser_declaration (parser=0x41831af0) at /Volumes/UFS_Partition/
pinskia/src/fsf/gcc-clean/src/gcc/cp/parser.c:5992
#8  0x0005aabc in cp_parser_declaration_seq_opt (parser=0x41831af0) at /Volumes/
UFS_Partition/pinskia/src/fsf/gcc-clean/src/gcc/cp/parser.c:5901
#9  0x0005ac98 in c_parse_file () at /Volumes/UFS_Partition/pinskia/src/fsf/gcc-clean/src/gcc/
cp/parser.c:2163
#10 0x000b1034 in c_common_parse_file (set_yydebug=1099111712) at /Volumes/UFS_Partition/
pinskia/src/fsf/gcc-clean/src/gcc/c-opts.c:1226
#11 0x003115f4 in toplev_main (argc=4453932, argv=0x41831af0) at /Volumes/UFS_Partition/
pinskia/src/fsf/gcc-clean/src/gcc/toplev.c:1830
#12 0x000028a0 in _start (argc=15, argv=0xbffffba0, envp=0xbffffbe0) at /SourceCache/Csu/
Csu-46/crt.c:267
#13 0x00002714 in start ()

Also can reproduce with this on i686-pc-linux:
register int foo __asm__ ("ebx");

>From Phil's regression hunter with that testcase: Search converges between 2003-08-18-trunk 
(#376) and 2003-08-19-trunk (#377).

You just need the option -funit-a-at-time flag to reproduce this.

[Bug c++/12624] [3.4 Regression] [unit-at-a-time] ICE in notice_global_symbol

[wilson at specifixinc dot com]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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



------- Additional Comments From wilson at specifixinc dot com  2003-10-23 21:45 -------
Subject: Re:  [3.4 Regression] [unit-at-a-time] ICE in notice_global_symbol

pinskia at gcc dot gnu dot org wrote:
> From Phil's regression hunter with that testcase: Search converges between 2003-08-18-trunk 
> (#376) and 2003-08-19-trunk (#377).

That would be Jan's patch that created the notice_global_symbol 
function.  We already knew the problem was with that code.  It shouldn't 
be used for global register variables.

I see two obvious ways to fix the problem.  We can check for the 
DECL_REGISTER bit, or we can check that DECL_RTL contains a MEM of a 
SYMBOL_REF.  I'm leaning towards the former.  That is the one I plan to 
test today with an ia64-linux bootstrap.

[Bug c++/12624] [3.4 Regression] [unit-at-a-time] ICE in notice_global_symbol

[cvs-commit at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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



------- Additional Comments From cvs-commit at gcc dot gnu dot org  2003-10-24 07:39 -------
Subject: Bug 12624

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	hubicka@gcc.gnu.org	2003-10-24 07:39:26

Modified files:
	gcc            : ChangeLog varasm.c 

Log message:
	PR c++/12624
	* varasm.c (notice_global_symbol): Disqualify global registers.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/ChangeLog.diff?cvsroot=gcc&r1=2.1537&r2=2.1538
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/gcc/varasm.c.diff?cvsroot=gcc&r1=1.393&r2=1.394

[Bug c++/12624] [3.4 Regression] [unit-at-a-time] ICE in notice_global_symbol

[pinskia at gcc dot gnu dot org]

PLEASE REPLY TO gcc-bugzilla@gcc.gnu.org ONLY, *NOT* gcc-bugs@gcc.gnu.org.

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


pinskia at gcc dot gnu dot org changed:

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


------- Additional Comments From pinskia at gcc dot gnu dot org  2003-10-24 16:12 -------
Fixed for 3.4.

[Bug c++/12624] [3.4 Regression] [unit-at-a-time] ICE in notice_global_symbol

[cvs-commit at gcc dot gnu dot org]

------- Additional Comments From cvs-commit at gcc dot gnu dot org  2005-04-29 06:11 -------
Subject: Bug 12624

CVSROOT:	/cvs/gcc
Module name:	gcc
Changes by:	mkoch@gcc.gnu.org	2005-04-29 06:11:28

Modified files:
	libjava        : ChangeLog 
	libjava/java/beans: FeatureDescriptor.java Introspector.java 

Log message:
	2005-04-29  Robert Schuster  <thebohemian@gmx.net>
	
	* java/beans/FeatureDescriptor.java:
	(getShortDescription): Implemented fallback mechanism and fixed
	documentation (fixes bug #12637).
	(getDisplayName): Dito.
	
	2005-04-29  Robert Schuster  <thebohemian@gmx.net>
	
	* java/beans/Introspector.java: Fixed bug #12624, BeanDescriptors
	will now be set correctly.
	(flushCaches): Now flushes all cached intermediate data.

Patches:
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/ChangeLog.diff?cvsroot=gcc&r1=1.3574&r2=1.3575
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/java/beans/FeatureDescriptor.java.diff?cvsroot=gcc&r1=1.5&r2=1.6
http://gcc.gnu.org/cgi-bin/cvsweb.cgi/gcc/libjava/java/beans/Introspector.java.diff?cvsroot=gcc&r1=1.8&r2=1.9



-- 


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