From mboxrd@z Thu Jan  1 00:00:00 1970
From: John Wehle <john@feith.com>
To: rth@redhat.com
Cc: gcc@gcc.gnu.org, pfeifer@dbai.tuwien.ac.at, lucier@math.purdue.edu
Subject: Re: Unreviewed patch (was: another bootstrap failure onsparc-sun-solaris28 in cp/xref.o)
Date: Thu, 27 Sep 2001 14:59:00 -0000
Message-id: <200109272158.f8RLw7002365@jwlab.FEITH.COM>
X-SW-Source: 2001-09/msg01149.html

>On Thu, Sep 27, 2001 at 04:53:35PM -0400, John Wehle wrote:
>> When compiling libobjc/linking.m the compiler outputs ".uaxword" in
>> order to output a double int.
>
> But why?  I wouldn't have thought there would be any any double ints
> in debug info in 32-bit mode.

The traceback (with my patch installed) is:

#0  assemble_integer (x=0x39a400, size=8, align=32, force=0)
    at ../../../../../src/GNU/gcc-current/gcc/varasm.c:1966
#1  0x21d3c8 in output_constant (exp=0x4aa6c0, size=8, align=32)
    at ../../../../../src/GNU/gcc-current/gcc/varasm.c:4487
#2  0x21e380 in output_constructor (exp=0x4242d0, size=24, align=32)
    at ../../../../../src/GNU/gcc-current/gcc/varasm.c:4751
#3  0x21d520 in output_constant (exp=0x478140, size=24, align=32)
    at ../../../../../src/GNU/gcc-current/gcc/varasm.c:4535
#4  0x1fe420 in rest_of_decl_compilation (decl=0x424348, asmspec=0x0, 
    top_level=1, at_end=0)
    at ../../../../../src/GNU/gcc-current/gcc/toplev.c:2535
#5  0x55844 in finish_decl (decl=0x424348, init=0x35ec00, 
    asmspec_tree=0x32b0a8)
    at ../../../../../src/GNU/gcc-current/gcc/c-decl.c:3627
#6  0x3aa28 in build_selector_translation_table ()
    at ../../../../../src/GNU/gcc-current/gcc/objc/objc-act.c:2238

which leads me to believe that problem compiling linking.m isn't
due to debug information.  UNALIGNED_DOUBLE_INT_ASM_OP is also
used by assemble_integer in addition to dw2_asm_output_* so it is
used anytime an unaligned double int is assembled.

I believed merely changing UNALIGNED_DOUBLE_INT_ASM_OP resulted
in the bootstrap failing at a different spot, however I don't
have the details at head.  I may have simply tried undefining it
and didn't try (TARGET_ARCH64 ? "\t.uaxword\t" : NULL) without
the changes to dwarf2asm.c.  I'll revisit this later today / tomorrow.

> Yes, and makes -dA output really ugly.

Perhaps assemble_integer can be improved to provide prettier output?
It doesn't seem desirable to have different routines to do the same
thing.

-- John
-------------------------------------------------------------------------
|   Feith Systems  |   Voice: 1-215-646-8000  |  Email: john@feith.com  |
|    John Wehle    |     Fax: 1-215-540-5495  |                         |
-------------------------------------------------------------------------