From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from rock.gnat.com (rock.gnat.com [205.232.38.15]) by sourceware.org (Postfix) with ESMTP id 0AA633858D34 for ; Tue, 2 Jun 2020 11:29:25 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 0AA633858D34 Authentication-Results: sourceware.org; dmarc=none (p=none dis=none) header.from=adacore.com Authentication-Results: sourceware.org; spf=pass smtp.mailfrom=oliva@adacore.com Received: from localhost (localhost.localdomain [127.0.0.1]) by filtered-rock.gnat.com (Postfix) with ESMTP id D36481179CC; Tue, 2 Jun 2020 07:29:24 -0400 (EDT) X-Virus-Scanned: Debian amavisd-new at gnat.com Received: from rock.gnat.com ([127.0.0.1]) by localhost (rock.gnat.com [127.0.0.1]) (amavisd-new, port 10024) with LMTP id 3zHJw5ZJzMJD; Tue, 2 Jun 2020 07:29:24 -0400 (EDT) Received: from free.home (tron.gnat.com [IPv6:2620:20:4000:0:46a8:42ff:fe0e:e294]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by rock.gnat.com (Postfix) with ESMTPS id 8CEFC1179C1; Tue, 2 Jun 2020 07:29:20 -0400 (EDT) Received: from livre.home (livre.home [172.31.160.2]) by free.home (8.15.2/8.15.2) with ESMTPS id 052BT3dd713240 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Tue, 2 Jun 2020 08:29:03 -0300 From: Alexandre Oliva To: Anthony Green Cc: Hans-Peter Nilsson via Gcc-patches , Hans-Peter Nilsson Subject: Re: Broken build Organization: Free thinker, does not speak for AdaCore References: <202005270104.04R14S5x009608@ignucius.se.axis.com> <87k10wj3kg.fsf@hack.labdroid.net> Errors-To: aoliva@lxoliva.fsfla.org Date: Tue, 02 Jun 2020 08:29:03 -0300 In-Reply-To: <87k10wj3kg.fsf@hack.labdroid.net> (Anthony Green's message of "Wed, 27 May 2020 20:39:11 -0400") Message-ID: User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain X-Scanned-By: MIMEDefang 2.84 X-Spam-Status: No, score=-11.8 required=5.0 tests=BAYES_00, GIT_PATCH_0, JMQ_SPF_NEUTRAL, KAM_DMARC_STATUS, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 02 Jun 2020 11:29:27 -0000 Hello, Anthony, H-P, On May 27, 2020, Anthony Green wrote: > Hans-Peter Nilsson via Gcc-patches writes: >> And here's an improper bug report. >> >> One of the commits between cfdff3eeb90..5c8344e7289 caused every >> single *linked* test to fail for cris-elf, like: > I can confirm that the moxie-elf test cases don't link either. I believe the problem reported by Hans-Peter was fixed in the follow up I put in the other day, but I happened to stumble across more recent test results you've posted, and it looks like you're still running into problems. > It looks like setting ldscript in the board description file doesn't > work. In my case that means "-Tsim.ld" isn't being passed through and > we can't link anymore. Here's moxie-sim.exp: > https://github.com/moxielogic/moxie-test-gcc/blob/7c707e187f101922e3ef7f6e23dbbd1890f9e8dd/moxie-sim.exp#L42 Looking more closely, I realized I used ldscripts instead of ldscript in the recent changes. That would explain at least the outputs.exp failures. Would you please let me know whether the patch below brings you back to the state before the revamp patch went in? Failing that, could you possibly share a gcc.log with me? Thanks in advance, diff --git a/gcc/testsuite/gcc.misc-tests/outputs.exp b/gcc/testsuite/gcc.misc-tests/outputs.exp index c3c6c2d..06a32db 100644 --- a/gcc/testsuite/gcc.misc-tests/outputs.exp +++ b/gcc/testsuite/gcc.misc-tests/outputs.exp @@ -48,7 +48,7 @@ set skip_lto ![check_effective_target_lto] # We do not compile to an executable, because that requires naming an output. set link_options "" set dest [target_info name] -foreach i { ldflags libs ldscripts } { +foreach i { ldflags libs ldscript } { if {[board_info $dest exists $i]} { set skip "" foreach opt [split [board_info $dest $i]] { diff --git a/gcc/testsuite/lib/gcc-defs.exp b/gcc/testsuite/lib/gcc-defs.exp index d591cb3..87eeb7d 100644 --- a/gcc/testsuite/lib/gcc-defs.exp +++ b/gcc/testsuite/lib/gcc-defs.exp @@ -287,7 +287,7 @@ proc dg-additional-files { args } { set gcc_adjusted_linker_flags 0 -# Add -Wl, before any file names in ldflags, libs, and ldscripts, so +# Add -Wl, before any file names in ldflags, libs, and ldscript, so # that default object files or libraries do not change the names of # gcc auxiliary outputs. @@ -300,7 +300,7 @@ proc gcc_adjust_linker_flags {} { if {![is_remote host]} { set dest [target_info name] - foreach i { ldflags libs ldscripts } { + foreach i { ldflags libs ldscript } { if {[board_info $dest exists $i]} { set opts [board_info $dest $i] set nopts {} -- Alexandre Oliva, freedom fighter he/him https://FSFLA.org/blogs/lxo/ Free Software Evangelist Stallman was right, but he's left :( GNU Toolchain Engineer Live long and free, and prosper ethically