From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from us-smtp-1.mimecast.com (us-smtp-delivery-1.mimecast.com [207.211.31.120]) by sourceware.org (Postfix) with ESMTP id 65BBC3945C23 for ; Wed, 22 Apr 2020 18:04:41 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.3.2 sourceware.org 65BBC3945C23 Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-359-iUeLV_NEOBKtouRmynNvNw-1; Wed, 22 Apr 2020 14:04:22 -0400 X-MC-Unique: iUeLV_NEOBKtouRmynNvNw-1 Received: from smtp.corp.redhat.com (int-mx06.intmail.prod.int.phx2.redhat.com [10.5.11.16]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id F1F41801E7E; Wed, 22 Apr 2020 18:04:20 +0000 (UTC) Received: from ovpn-113-172.phx2.redhat.com (ovpn-113-172.phx2.redhat.com [10.3.113.172]) by smtp.corp.redhat.com (Postfix) with ESMTP id 2CDE25C557; Wed, 22 Apr 2020 18:04:20 +0000 (UTC) Message-ID: <7d67eb66e65632ab1fce66ba07a8aa3e6090953d.camel@redhat.com> Subject: Re: [PATCH v5 GCC] libffi/test: Fix compilation for build sysroot From: Jeff Law Reply-To: law@redhat.com To: "Maciej W. Rozycki" , gcc-patches@gcc.gnu.org Cc: Mike Stump , Chung-Lin Tang , Ian Lance Taylor , libffi-discuss@sourceware.org Date: Wed, 22 Apr 2020 12:04:19 -0600 In-Reply-To: References: Organization: Red Hat User-Agent: Evolution 3.34.4 (3.34.4-1.fc31) MIME-Version: 1.0 X-Scanned-By: MIMEDefang 2.79 on 10.5.11.16 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit X-Spam-Status: No, score=-4.9 required=5.0 tests=BAYES_00, DKIMWL_WL_HIGH, DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, DKIM_VALID_EF, RCVD_IN_DNSWL_NONE, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=unavailable autolearn_force=no version=3.4.2 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on server2.sourceware.org X-BeenThere: libffi-discuss@sourceware.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Libffi-discuss mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 22 Apr 2020 18:04:43 -0000 On Mon, 2020-04-20 at 14:18 +0100, Maciej W. Rozycki wrote: > Fix a problem with the libffi testsuite using a method to determine the > compiler to use resulting in the tool being different from one the > library has been built with, and causing a catastrophic failure from the > inability to actually choose any compiler at all in a cross-compilation > configuration. > > Address this problem by providing a DejaGNU configuration file defining > the compiler to use, via the CC_FOR_TARGET TCL variable, set from $CC by > autoconf, which will have all the required options set for the target > compiler to build executables in the environment configured, removing > failures like: > > FAIL: libffi.call/closure_fn0.c -W -Wall -Wno-psabi -O0 (test for excess > errors) > Excess errors: > default_target_compile: No compiler to compile with > UNRESOLVED: libffi.call/closure_fn0.c -W -Wall -Wno-psabi -O0 compilation > failed to produce executable > > and bringing overall test results for the `riscv64-linux-gnu' target > (here with the `x86_64-linux-gnu' host and RISC-V QEMU in the Linux user > emulation mode as the target board) from: > > === libffi Summary === > > # of unexpected failures 708 > # of unresolved testcases 708 > # of unsupported tests 30 > > to: > > === libffi Summary === > > # of expected passes 1934 > # of unsupported tests 28 > > This is a combined backport of the relevant parts of upstream libffi > changes as follows: > > - commit 8308984e479e ("[PATCH] Make sure we're running dejagnu tests > with the right compiler."), > > - commit 2d9b3939751b ("[PATCH] Fix for closures with sunpro compiler"), > > - commit 0c3824702d3d ("[PATCH] Always set CC_FOR_TARGET for dejagnu, to > make the testsuite respect $CC"), > > - commit 7d698125b1f0 ("[PATCH] Use the proper C++ compiler to run C++ > tests"), > > - commit 6b6df1a7bb37 ("[PATCH] Adds `local.exp` to CLEANFILES"), > > - commit 6cf0dea78a5a ("[PATCH] Change CLEANFILES to DISTCLEANFILES") > > libffi/ > * Makefile.am (DISTCLEANFILES): New variable. > * configure.ac: Produce `local.exp'. > * Makefile.in: Regenerate. > * configure: Regenerate. > * testsuite/Makefile.am (EXTRA_DEJAGNU_SITE_CONFIG): New > variable. > * testsuite/Makefile.in: Regenerate. OK jeff >