From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 79619 invoked by alias); 11 Nov 2019 18:12:54 -0000 Mailing-List: contact libffi-discuss-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: libffi-discuss-owner@sourceware.org Received: (qmail 79602 invoked by uid 89); 11 Nov 2019 18:12:54 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-8.9 required=5.0 tests=AWL,BAYES_00,GIT_PATCH_2,GIT_PATCH_3,KAM_ASCII_DIVIDERS,KAM_SHORT autolearn=ham version=3.3.1 spammy= X-HELO: esa2.hgst.iphmx.com Received: from esa2.hgst.iphmx.com (HELO esa2.hgst.iphmx.com) (68.232.143.124) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 11 Nov 2019 18:12:52 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com; t=1573495980; x=1605031980; h=date:from:to:cc:subject:message-id:mime-version; bh=AEUr/ZNTCuy1XfLYIiw5QJxmrMw8RTbQvg7YgqLEC90=; b=EteiHPQUrqxqYyRkkhqGq1IJ56SYtPU8ugqgcuf4FFrEom6sIalVTjQ1 29hN1ne9oHDu0dH+c6O9q2Dh4s5eXns+tOCiL53uoEGsYQBP8bxQfEQDk bVw0Mgkqb2TpaQE0zBuUIM4hGCKJgKwtKxnqDtL3t3x9RF/J1y7vUFSVz Q+lYtIfkthP69/lJzp/VWx5iFi2xH9OXvjDw918Gp07w7KzpMigxeL63+ lwHKFWFgtzyF3Th42dS3fnjKmjAqJGfkGOk0vZ8I1LnDV0tcprhXu+S15 zcnbtbk1yUV/OzNR0eXQImRUHdd2Diw6nVEzX7nf+FRNzURGZPj6Hqz+W g==; IronPort-SDR: YrgMJRI2+eZh31oNrEJ4nmxUf6ZVXyUpJPbnAr1Z6MmWgM+KaET9AVZtS8hkVNst06a7m9bzeq adBGjLf0PePEPUa7+uK+cx7/4VLnsbLVX+yUXHxUlvvzfLMOnqUVruNZ5QM52LUwFP73q2GZPU p20zA5qP7Ky94uCVIoqRNiOQHp6rMouvKvR+mPmre2zv/UpznTg4rT12Ohwk1A8PTwabvn3iGu AROdaDNTHbGbuQ4pWv/sTD/ZswPN/FD/1vSAPGrCe1WMXLysuu0hWnzYTGtZ4cD9nSA/T7hZhZ bXM= X-URL-LookUp-ScanningError: 1 Received: from h199-255-45-15.hgst.com (HELO uls-op-cesaep02.wdc.com) ([199.255.45.15]) by ob1.hgst.iphmx.com with ESMTP; 12 Nov 2019 02:12:52 +0800 IronPort-SDR: 0s/Fvo38dNAZrtEj9M8BjFuyf+c9O8GJFfFXDZskFbRl5ESHGAkr3L+U2FF6cQeoDZRw1Uu0ty UDdW/SvhruSKcBTvAs3twUzFDr+5TGquwYSAPzAmTLvqKEdg+5C0QNsidW2M9epRZ/K8GN9bIW J5xrslRq3dQfEKCUh2JnPlsj+HUG6kmonRsGfqKPArX0gf86Hnd16/c0DCg6w3UlY6aoWxfKI1 TA2gEQuR/p4t6xoP6uoin5lzashEjdX+ayBsH+SsDSQkA6YKhNFBSlbMArp4acxOzClvdRmQsR SRFCS61fjdWVGjk/KgRVs6IL Received: from uls-op-cesaip01.wdc.com ([10.248.3.36]) by uls-op-cesaep02.wdc.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Nov 2019 10:07:50 -0800 IronPort-SDR: an78HJeoZSKsbD5WTBfDq28ab9a1zUKYSx5m0k+9KAx/XUKjxDgISiRa6oEvf/23jfuzejY/9u lSnRAPatvzD0EaSPosz2fwmM8RYcmr44RxD9vGQ9HkU3MwVQQorGSFpo7DIYJGMrMTn/0twXLp i7YByhiIDCf19ZTYCJh1rEGNyCXYrzl6YwDhxqi5482DxJtS99q28QkrWFZymsH97dezphJAtt UmAl8fHHl1CHS0YwM7SeJMl2sPgQdIIPOfuiwiHUVHxM0xStT4N4ujZp2xx91EhK93jbe1Lai8 YsE= WDCIronportException: Internal Received: from unknown (HELO redsun52) ([10.149.66.28]) by uls-op-cesaip01.wdc.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 11 Nov 2019 10:12:46 -0800 Date: Mon, 11 Nov 2019 18:12:00 -0000 From: "Maciej W. Rozycki" To: gcc-patches@gcc.gnu.org cc: libffi-discuss@sourceware.org, golang-dev@googlegroups.com Subject: [PATCH 3/4] libgo/test: Fix compilation for build sysroot Message-ID: User-Agent: Alpine 2.21 (LFD 202 2017-01-01) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-SW-Source: 2019/txt/msg00068.txt.bz2 Fix a problem with the libgo 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 lack of a suitable `--sysroot=' option where the `--with-build-sysroot=' configuration option has been used to build the compiler resulting in the inability to link executables. Address this problem by providing a DejaGNU configuration file defining the compiler to use, via the GOC_UNDER_TEST TCL variable, set from $GOC by autoconf, which will have all the required options set for the target compiler to build executables in the environment configured, removing failures like: .../bin/riscv64-linux-gnu-ld: cannot find crt1.o: No such file or directory .../bin/riscv64-linux-gnu-ld: cannot find -lm .../bin/riscv64-linux-gnu-ld: cannot find -lc collect2: error: ld returned 1 exit status compiler exited with status 1 No summary comparison, because the libgo testsuite does not provide one in this configuration for some reason, however this change improves 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 0 PASSes and 159 FAILs to 133 PASSes and 26 FAILs. libgo/ * configure.ac: Add testsuite/libgo-test-support.exp to output files. * configure: Regenerate. * testsuite/libgo-test-support.exp.in: New file. * testsuite/Makefile.am (EXTRA_DEJAGNU_SITE_CONFIG): New variable. * testsuite/Makefile.in: Regenerate. * testsuite/lib/libgo.exp: Don't override GOC_UNDER_TEST previously set. --- libgo/configure | 3 ++- libgo/configure.ac | 2 +- libgo/testsuite/Makefile.am | 2 ++ libgo/testsuite/Makefile.in | 5 ++++- libgo/testsuite/lib/libgo.exp | 13 ++++++++----- libgo/testsuite/libgo-test-support.exp.in | 1 + 6 files changed, 18 insertions(+), 8 deletions(-) gcc-test-libgo-gcc-under-test.diff Index: gcc/libgo/configure =================================================================== --- gcc.orig/libgo/configure +++ gcc/libgo/configure @@ -15850,7 +15850,7 @@ else multilib_arg= fi -ac_config_files="$ac_config_files Makefile testsuite/Makefile" +ac_config_files="$ac_config_files Makefile testsuite/Makefile testsuite/libgo-test-support.exp" ac_config_commands="$ac_config_commands default" @@ -17031,6 +17031,7 @@ do "libtool") CONFIG_COMMANDS="$CONFIG_COMMANDS libtool" ;; "Makefile") CONFIG_FILES="$CONFIG_FILES Makefile" ;; "testsuite/Makefile") CONFIG_FILES="$CONFIG_FILES testsuite/Makefile" ;; + "testsuite/libgo-test-support.exp") CONFIG_FILES="$CONFIG_FILES testsuite/libgo-test-support.exp" ;; "default") CONFIG_COMMANDS="$CONFIG_COMMANDS default" ;; *) as_fn_error $? "invalid argument: \`$ac_config_target'" "$LINENO" 5;; Index: gcc/libgo/configure.ac =================================================================== --- gcc.orig/libgo/configure.ac +++ gcc/libgo/configure.ac @@ -877,7 +877,7 @@ else multilib_arg= fi -AC_CONFIG_FILES(Makefile testsuite/Makefile) +AC_CONFIG_FILES(Makefile testsuite/Makefile testsuite/libgo-test-support.exp) AC_CONFIG_COMMANDS([default], [if test -n "$CONFIG_FILES"; then Index: gcc/libgo/testsuite/Makefile.am =================================================================== --- gcc.orig/libgo/testsuite/Makefile.am +++ gcc/libgo/testsuite/Makefile.am @@ -11,6 +11,8 @@ RUNTEST = `if [ -f $(top_srcdir)/../deja echo $(top_srcdir)/../dejagnu/runtest ; \ else echo runtest; fi` +EXTRA_DEJAGNU_SITE_CONFIG = libgo-test-support.exp + # When running the tests we set GCC_EXEC_PREFIX to the install tree so that # files that have already been installed there will be found. The -B option # overrides it, so use of GCC_EXEC_PREFIX will not result in using GCC files Index: gcc/libgo/testsuite/Makefile.in =================================================================== --- gcc.orig/libgo/testsuite/Makefile.in +++ gcc/libgo/testsuite/Makefile.in @@ -106,7 +106,7 @@ am__configure_deps = $(am__aclocal_m4_de DIST_COMMON = $(srcdir)/Makefile.am mkinstalldirs = $(SHELL) $(top_srcdir)/../mkinstalldirs CONFIG_HEADER = $(top_builddir)/config.h -CONFIG_CLEAN_FILES = +CONFIG_CLEAN_FILES = libgo-test-support.exp CONFIG_CLEAN_VPATH_FILES = AM_V_P = $(am__v_P_@AM_V@) am__v_P_ = $(am__v_P_@AM_DEFAULT_V@) @@ -299,6 +299,7 @@ RUNTEST = `if [ -f $(top_srcdir)/../deja echo $(top_srcdir)/../dejagnu/runtest ; \ else echo runtest; fi` +EXTRA_DEJAGNU_SITE_CONFIG = libgo-test-support.exp # When running the tests we set GCC_EXEC_PREFIX to the install tree so that # files that have already been installed there will be found. The -B option @@ -338,6 +339,8 @@ $(top_srcdir)/configure: @MAINTAINER_MOD $(ACLOCAL_M4): @MAINTAINER_MODE_TRUE@ $(am__aclocal_m4_deps) cd $(top_builddir) && $(MAKE) $(AM_MAKEFLAGS) am--refresh $(am__aclocal_m4_deps): +libgo-test-support.exp: $(top_builddir)/config.status $(srcdir)/libgo-test-support.exp.in + cd $(top_builddir) && $(SHELL) ./config.status $(subdir)/$@ mostlyclean-libtool: -rm -f *.lo Index: gcc/libgo/testsuite/lib/libgo.exp =================================================================== --- gcc.orig/libgo/testsuite/lib/libgo.exp +++ gcc/libgo/testsuite/lib/libgo.exp @@ -14,13 +14,16 @@ # along with GCC; see the file COPYING3. If not see # . -set gccdir [lookfor_file $tool_root_dir gcc/libgcc.a] -if {$gccdir != ""} { - set gccdir [file dirname $gccdir] -} -set GOC_UNDER_TEST "$gccdir/gccgo -B$gccdir/" set TESTING_IN_BUILD_TREE 1 +if ![info exists GOC_UNDER_TEST] then { + set gccdir [lookfor_file $tool_root_dir gcc/libgcc.a] + if {$gccdir != ""} { + set gccdir [file dirname $gccdir] + } + set GOC_UNDER_TEST "$gccdir/gccgo -B$gccdir/" +} + if [info exists GOTEST_TMPDIR] { set tmpdir $GOTEST_TMPDIR } Index: gcc/libgo/testsuite/libgo-test-support.exp.in =================================================================== --- /dev/null +++ gcc/libgo/testsuite/libgo-test-support.exp.in @@ -0,0 +1 @@ +set GOC_UNDER_TEST {@GOC@}