From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 24572 invoked by alias); 6 Mar 2014 08:18:51 -0000 Mailing-List: contact binutils-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: binutils-owner@sourceware.org Received: (qmail 24558 invoked by uid 89); 6 Mar 2014 08:18:50 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.6 required=5.0 tests=AWL,BAYES_00 autolearn=ham version=3.3.2 X-HELO: smtp.eu.adacore.com Received: from mel.act-europe.fr (HELO smtp.eu.adacore.com) (194.98.77.210) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-GCM-SHA384 encrypted) ESMTPS; Thu, 06 Mar 2014 08:18:47 +0000 Received: from localhost (localhost [127.0.0.1]) by filtered-smtp.eu.adacore.com (Postfix) with ESMTP id 38B3126EF0C0; Thu, 6 Mar 2014 09:18:44 +0100 (CET) Received: from smtp.eu.adacore.com ([127.0.0.1]) by localhost (smtp.eu.adacore.com [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id HK7rpFahBn4m; Thu, 6 Mar 2014 09:18:44 +0100 (CET) Received: from ulanbator.act-europe.fr (ulanbator.act-europe.fr [10.10.1.67]) (using TLSv1 with cipher ECDHE-RSA-AES256-SHA (256/256 bits)) (No client certificate requested) by smtp.eu.adacore.com (Postfix) with ESMTPSA id 1242F26EF085; Thu, 6 Mar 2014 09:18:44 +0100 (CET) Content-Type: text/plain; charset=us-ascii Mime-Version: 1.0 (Mac OS X Mail 7.2 \(1874\)) Subject: Re: PR ld/16643: ARM: BFD_ASSERT when only ref to a GC'd symbol is in a stripped section From: Tristan Gingold In-Reply-To: Date: Thu, 06 Mar 2014 08:18:00 -0000 Cc: "binutils@sourceware.org" Content-Transfer-Encoding: quoted-printable Message-Id: <6445F8EC-8A18-474F-B854-5827A08925DE@adacore.com> References: <20140228002516.GE14922@bubble.grove.modra.org> To: Roland McGrath X-IsSubscribed: yes X-SW-Source: 2014-03/txt/msg00043.txt.bz2 On 05 Mar 2014, at 19:07, Roland McGrath wrote: > ping. > Still looking for maintainer approval to commit my test case. > Still looking for release maintainer approval to backport fix+test to 2.2= 4. OK for 2.24 once approved on trunk. Tristan. >=20 > On Fri, Feb 28, 2014 at 9:44 AM, Roland McGrath wro= te: >> On Thu, Feb 27, 2014 at 4:25 PM, Alan Modra wrote: >>> Here you go, this cures your testcase. >>=20 >> Great! Thanks a lot. >>=20 >> OK to commit the test case now? >>=20 >> OK to backport both fix and test to 2.24? >>=20 >>=20 >> Thanks, >> Roland >>=20 >>=20 >> ld/testsuite/ >> 2014-02-28 Roland McGrath >>=20 >> PR ld/16643 >> * ld-arm/gc-hidden-strip.d: New file. >> * ld-arm/gc-hidden-strip-unused.s: New file. >> * ld-arm/gc-hidden-strip-main.s: New file. >> * ld-arm/arm-elf.exp: Run the new test. >>=20 >> --- a/ld/testsuite/ld-arm/arm-elf.exp >> +++ b/ld/testsuite/ld-arm/arm-elf.exp >> @@ -842,3 +842,4 @@ if { ![istarget "arm*-*-nacl*"] } { >> } >> run_dump_test "unresolved-2" >> run_dump_test "gc-hidden-1" >> +run_dump_test "gc-hidden-strip" >> --- /dev/null >> +++ b/ld/testsuite/ld-arm/gc-hidden-strip-main.s >> @@ -0,0 +1,6 @@ >> + .text >> + .globl foo >> + .type foo, %function >> +foo: >> + bx lr >> + .size foo, . - foo >> --- /dev/null >> +++ b/ld/testsuite/ld-arm/gc-hidden-strip-unused.s >> @@ -0,0 +1,11 @@ >> + .section .data.unused_item,"aw",%progbits >> + .p2align 2 >> + .global unused_item >> + .hidden unused_item >> + .type unused_item, %object >> + .size unused_item, 4 >> +unused_item: >> + .word 1 >> + >> + .section .debug_blah,"",%progbits >> + .word unused_item >> --- /dev/null >> +++ b/ld/testsuite/ld-arm/gc-hidden-strip.d >> @@ -0,0 +1,15 @@ >> +#source: gc-hidden-strip-main.s >> +#source: gc-hidden-strip-unused.s >> +#ld: --gc-sections --shared --strip-all >> +#objdump: -RT >> +# This test is only valid on ELF based ports. >> +# not-target: *-*-*coff *-*-pe *-*-wince *-*-*aout* *-*-netbsd *-*-risc= ix* >> + >> +# See PR ld/16643: the only reference to a GC'd symbol is in a stripped >> +# section, trigging a BFD_ASSERT. >> + >> +.*: file format elf32-.* >> + >> +DYNAMIC SYMBOL TABLE: >> +0+[0-9a-f]+ l\s+d\s+\.text\s+0+\s+\.text >> +#pass