From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 607 invoked by alias); 23 Nov 2004 22:20:12 -0000 Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Archive: List-Post: List-Help: Sender: gcc-bugs-owner@gcc.gnu.org Received: (qmail 576 invoked by alias); 23 Nov 2004 22:20:06 -0000 Date: Tue, 23 Nov 2004 22:20:00 -0000 Message-ID: <20041123222006.574.qmail@sourceware.org> From: "zack at codesourcery dot com" To: gcc-bugs@gcc.gnu.org In-Reply-To: <20041013220543.17982.schwab@suse.de> References: <20041013220543.17982.schwab@suse.de> Reply-To: gcc-bugzilla@gcc.gnu.org Subject: [Bug middle-end/17982] [4.0 regression] asm declaration ignored due to conflict with previous rename X-Bugzilla-Reason: CC X-SW-Source: 2004-11/txt/msg02743.txt.bz2 List-Id: ------- Additional Comments From zack at codesourcery dot com 2004-11-23 22:19 ------- Subject: Re: [4.0 regression] asm declaration ignored due to conflict with previous rename "hjl at lucon dot org" writes: > ------- Additional Comments From hjl at lucon dot org 2004-11-23 21:13 ------- > Does this patch > > --- gcc/c-pragma.c.rename 2004-11-09 12:03:42.000000000 -0800 > +++ gcc/c-pragma.c 2004-11-23 13:03:26.020304351 -0800 > @@ -473,8 +473,11 @@ maybe_apply_renaming_pragma (tree decl, > return asmname; > > /* If the DECL_ASSEMBLER_NAME is already set, it does not change, > - but we may warn about a rename that conflicts. */ > - if (DECL_ASSEMBLER_NAME_SET_P (decl)) > + but we may warn about a rename that conflicts. > + FIXME: the DECL_ASSEMBLER_NAME can be set to DECL_NAME (decl) > + without renaming pragma nor asm declaration involved. */ This will not work. At this point ia64 has already emitted an external-reference directive for the old name into the assembly stream. The necessary fix is to prevent assemble_external from ever being called before EOF. zw -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=17982