From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <java-prs-return-11565-listarch-java-prs=gcc.gnu.org@gcc.gnu.org>
Received: (qmail 1950 invoked by alias); 2 May 2013 18:21:28 -0000
Mailing-List: contact java-prs-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <java-prs.gcc.gnu.org>
List-Subscribe: <mailto:java-prs-subscribe@gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/java-prs/>
List-Post: <mailto:java-prs@gcc.gnu.org>
List-Help: <mailto:java-prs-help@gcc.gnu.org>, <http://gcc.gnu.org/ml/#faqs>
Sender: java-prs-owner@gcc.gnu.org
Received: (qmail 1898 invoked by uid 48); 2 May 2013 18:21:28 -0000
From: "jakub at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: java-prs@gcc.gnu.org
Subject: [Bug libgcj/57074] gcc-4.8.0 libgcj regression on 32bit Power
 architecture
Date: Thu, 02 May 2013 18:21:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: libgcj
X-Bugzilla-Keywords: 
X-Bugzilla-Severity: critical
X-Bugzilla-Who: jakub at gcc dot gnu.org
X-Bugzilla-Status: NEW
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 4.8.1
X-Bugzilla-Changed-Fields: CC Target Milestone
Message-ID: <bug-57074-8172-taEZHpQ27X@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-57074-8172@http.gcc.gnu.org/bugzilla/>
References: <bug-57074-8172@http.gcc.gnu.org/bugzilla/>
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
Content-Type: text/plain; charset="UTF-8"
MIME-Version: 1.0
X-SW-Source: 2013-q2/txt/msg00014.txt.bz2


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

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |aph at gcc dot gnu.org,
                   |                            |tromey at gcc dot gnu.org
   Target Milestone|---                         |4.8.1

--- Comment #8 from Jakub Jelinek <jakub at gcc dot gnu.org> 2013-05-02 18:21:24 UTC ---
So, the decl in question seems to be created by:
#0  build_decl_stat (loc=13046123, code=VAR_DECL, name=0xf42cd7d0,
type=0xf7915840) at ../../gcc/tree.c:4166
#1  0x081bbf87 in gen_indirect_dispatch_tables (type=0xf7973f60) at
../../gcc/java/class.c:472
#2  0x08207f7b in parse_class_file () at ../../gcc/java/jcf-parse.c:1566
#3  0x082091bc in parse_zip_file_entries () at ../../gcc/java/jcf-parse.c:2101

and the DECL_INITIAL for it filled by:

#0  emit_symbol_table (name=0xf42cd7a8, the_table=0xf42cf5c0,
decl_table=0xf42cd938, the_syms_decl=the_syms_decl@entry=0xf42cf61c, 
the_array_element_type=the_array_element_type@entry=0xf790e720,
element_size=element_size@entry=1) at ../../gcc/java/class.c:2965
#1  0x081c7854 in make_class_data (type=0xf7973f60) at
../../gcc/java/class.c:2066
#2  0x081d13ca in finish_class () at ../../gcc/java/class.c:2255
#3  0x082081df in parse_class_file () at ../../gcc/java/jcf-parse.c:1661
#4  0x082091bc in parse_zip_file_entries () at ../../gcc/java/jcf-parse.c:2101

One bug seems to be
one_elt_array_domain_type = build_index_type (integer_one_node);
That name seems to be inconsistent with what it does, build_index_type
(integer_one_node); returns an index type with <min 0> <max 1>, i.e. two
element array.  One element array is build_index_type (integer_zero_node);.

And, either the individual decls need to be created with different types,
depending on how many elements they will have, or at least treated as C []
arrays and at least DECL_SIZE and DECL_SIZE_UNIT need to be updated when
setting DECL_INITIAL.

>>From quick skimming, emit_{symbol,assertion}_table are the only callers in
gcc/java/ of build_constructor that don't bother with sizing the array type
appropriately.