* [PATCH] Remove extra lib directory from --with-advance-toolchain @ 2019-10-03 17:08 Tulio Magno Quites Machado Filho 2019-10-03 23:32 ` Michael Meissner 2019-10-04 16:13 ` [PATCH] Remove extra lib directory from --with-advance-toolchain Jeff Law 0 siblings, 2 replies; 8+ messages in thread From: Tulio Magno Quites Machado Filho @ 2019-10-03 17:08 UTC (permalink / raw) To: gcc-patches; +Cc: segher, meissner Remove the extra -L directory so that user directories specified at build time have higher preference over the advance toolchain libraries. 2019-10-03 Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com> * config.gcc: Remove an extra -L from --with-advance-toolchain. --- gcc/config.gcc | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) diff --git a/gcc/config.gcc b/gcc/config.gcc index 481bc9586a7..d52b5cd2d41 100644 --- a/gcc/config.gcc +++ b/gcc/config.gcc @@ -4782,14 +4782,12 @@ case "${target}" in echo "#undef LINK_OS_EXTRA_SPEC32" echo "#define LINK_OS_EXTRA_SPEC32" \ "\"%(link_os_new_dtags)" \ - "-rpath $prefix/lib -rpath $at/lib" \ - "-L $prefix/lib -L $at/lib\"" + "-rpath $prefix/lib -rpath $at/lib\"" echo echo "#undef LINK_OS_EXTRA_SPEC64" echo "#define LINK_OS_EXTRA_SPEC64" \ "\"%(link_os_new_dtags)" \ - "-rpath $prefix/lib64 -rpath $at/lib64" \ - "-L $prefix/lib64 -L $at/lib64\"" + "-rpath $prefix/lib64 -rpath $at/lib64\"" echo echo "#undef LINK_OS_NEW_DTAGS_SPEC" echo "#define LINK_OS_NEW_DTAGS_SPEC" \ -- 2.14.5 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Remove extra lib directory from --with-advance-toolchain 2019-10-03 17:08 [PATCH] Remove extra lib directory from --with-advance-toolchain Tulio Magno Quites Machado Filho @ 2019-10-03 23:32 ` Michael Meissner 2019-10-04 21:32 ` [PATCHv2] Change the library search path when using --with-advance-toolchain Tulio Magno Quites Machado Filho 2019-10-04 16:13 ` [PATCH] Remove extra lib directory from --with-advance-toolchain Jeff Law 1 sibling, 1 reply; 8+ messages in thread From: Michael Meissner @ 2019-10-03 23:32 UTC (permalink / raw) To: Tulio Magno Quites Machado Filho; +Cc: gcc-patches, segher, meissner On Thu, Oct 03, 2019 at 02:08:16PM -0300, Tulio Magno Quites Machado Filho wrote: > Remove the extra -L directory so that user directories specified at > build time have higher preference over the advance toolchain libraries. > > 2019-10-03 Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com> > > * config.gcc: Remove an extra -L from --with-advance-toolchain. > --- > gcc/config.gcc | 6 ++---- > 1 file changed, 2 insertions(+), 4 deletions(-) > > diff --git a/gcc/config.gcc b/gcc/config.gcc > index 481bc9586a7..d52b5cd2d41 100644 > --- a/gcc/config.gcc > +++ b/gcc/config.gcc > @@ -4782,14 +4782,12 @@ case "${target}" in > echo "#undef LINK_OS_EXTRA_SPEC32" > echo "#define LINK_OS_EXTRA_SPEC32" \ > "\"%(link_os_new_dtags)" \ > - "-rpath $prefix/lib -rpath $at/lib" \ > - "-L $prefix/lib -L $at/lib\"" > + "-rpath $prefix/lib -rpath $at/lib\"" > echo > echo "#undef LINK_OS_EXTRA_SPEC64" > echo "#define LINK_OS_EXTRA_SPEC64" \ > "\"%(link_os_new_dtags)" \ > - "-rpath $prefix/lib64 -rpath $at/lib64" \ > - "-L $prefix/lib64 -L $at/lib64\"" > + "-rpath $prefix/lib64 -rpath $at/lib64\"" > echo > echo "#undef LINK_OS_NEW_DTAGS_SPEC" > echo "#define LINK_OS_NEW_DTAGS_SPEC" \ > -- > 2.14.5 > I did a spec build with a compiler created with the following configure options: --enable-languages=c,c++,fortran --enable-checking \ --enable-stage1-checking --enable-gnu-indirect-function \ --enable-plugin --enable-decimal-float --with-long-double-128 \ --enable-secureplt --enable-threads=posix --enable-__cxa_atexit \ --with-cpu=power8 \ --with-as=/home/meissner/fsf-install-ppc64le/pu-binutils/bin/as \ --with-ld=/home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld \ --with-gnu-as=/home/meissner/fsf-install-ppc64le/pu-binutils/bin/as \ --with-gnu-ld=/home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld \ --with-advance-toolchain=at12.0 \ --with-native-system-header-dir=/opt/at12.0/include \ --without-ppl --without-cloog --without-isl And then I built Spec 2006 and 2017 with my normal options on a power8 system running Ubuntu and an older set of host libraries. If I have this patch installed, it breaks linking some/all of the C++ and Fortran benchmarks, because it finds the host C++/Fortran libraries before the AT 12.0 libraries. Now, if I need to add more -I, -L, -Wl,-rpath,... options I can do it, but I suspect it will break anybody else that uses --with-advance-toolchain=<xxx> to completely avoid using the host libraries. Here is the information from the Spec 2017 cactuBSSN_r benchmark, which is a C++ benchmark: /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: Cactus/piraha/Call.o: in function `cctki_piraha::meval(cctki_piraha::smart_ptr<cctki_piraha::Group>)': /home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/507.cactuBSSN_r/build/build_base_base_64.0000/Cactus/piraha/Call.cc:420: undefined reference to `std::__cxx11::basic_ostringstream<char, std::char_traits<char>, std::allocator<char> >::basic_ostringstream()' /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: /home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/507.cactuBSSN_r/build/build_base_base_64.0000/Cactus/piraha/Call.cc:448: undefined reference to `std::__cxx11::basic_ostringstream<char, std::char_traits<char>, std::allocator<char> >::basic_ostringstream()' /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: /home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/507.cactuBSSN_r/build/build_base_base_64.0000/Cactus/piraha/Call.cc:462: undefined reference to `std::__cxx11::basic_ostringstream<char, std::char_traits<char>, std::allocator<char> >::basic_ostringstream()' /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: /home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/507.cactuBSSN_r/build/build_base_base_64.0000/Cactus/piraha/Call.cc:451: undefined reference to `std::__cxx11::basic_stringstream<char, std::char_traits<char>, std::allocator<char> >::basic_stringstream()' /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: /home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/507.cactuBSSN_r/build/build_base_base_64.0000/Cactus/piraha/Call.cc:478: undefined reference to `std::__cxx11::basic_ostringstream<char, std::char_traits<char>, std::allocator<char> >::basic_ostringstream()' /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: /home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/507.cactuBSSN_r/build/build_base_base_64.0000/Cactus/piraha/Call.cc:493: undefined reference to `std::__cxx11::basic_ostringstream<char, std::char_traits<char>, std::allocator<char> >::basic_ostringstream()' /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: /home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/507.cactuBSSN_r/build/build_base_base_64.0000/Cactus/piraha/Call.cc:487: undefined reference to `std::__cxx11::basic_ostringstream<char, std::char_traits<char>, std::allocator<char> >::basic_ostringstream()' /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: /home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/507.cactuBSSN_r/build/build_base_base_64.0000/Cactus/piraha/Call.cc:544: undefined reference to `std::__cxx11::basic_ostringstream<char, std::char_traits<char>, std::allocator<char> >::basic_ostringstream()' /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: /home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/507.cactuBSSN_r/build/build_base_base_64.0000/Cactus/piraha/Call.cc:602: undefined reference to `std::__cxx11::basic_ostringstream<char, std::char_traits<char>, std::allocator<char> >::basic_ostringstream()' /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: Cactus/piraha/Call.o:/home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/507.cactuBSSN_r/build/build_base_base_64.0000/Cactus/piraha/Call.cc:622: more undefined references to `std::__cxx11::basic_ostringstream<char, std::char_traits<char>, std::allocator<char> >::basic_ostringstream()' follow The C++ lines I used to build the benchmark .o files for C++ was: <...>/g++ -c -o Cactus/main/flesh.o -DSPEC -DSPEC_CPU -DNDEBUG \ -Iinclude -DCCODE -DSPEC_SUPPRESS_OPENMP \ -DCCTK_DISABLE_RESTRICT=1 -g -mlittle -save-temps=obj \ -ffast-math -Ofast -mcpu=power9 -mrecip -funroll-loops \ -mno-pointers-to-nested-functions -msave-toc-indirect -m64 \ -I/opt/at12.0/include \ -I/opt/at12.0/lib/gcc/powerpc64le-linux-gnu/8.3.1/include \ -DSPEC_LP64 Cactus/main/flesh.cc Similarly for Fortran, here is the Spec 2017 wrf_r benchmark, which is a Fortran benchmark: /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: diffwrf.fppized.o: in function `MAIN__': /home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/521.wrf_r/build/build_base_base_64.0000/diffwrf.fppized.f90:1626: undefined reference to `_gfortran_os_error_at' /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: /home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/521.wrf_r/build/build_base_base_64.0000/diffwrf.fppized.f90:1729: undefined reference to `_gfortran_os_error_at' /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: /home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/521.wrf_r/build/build_base_base_64.0000/diffwrf.fppized.f90:1730: undefined reference to `_gfortran_os_error_at' /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: ESMF_AlarmClock.fppized.o: in function `__esmf_alarmclockmod_MOD_esmf_alarmcreate': /home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/521.wrf_r/build/build_base_base_64.0000/ESMF_AlarmClock.fppized.f90:118: undefined reference to `_gfortran_os_error_at' /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: ESMF_Clock.fppized.o: in function `__esmf_clockmod_MOD_esmf_clockcreate': /home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/521.wrf_r/build/build_base_base_64.0000/ESMF_Clock.fppized.f90:303: undefined reference to `_gfortran_os_error_at' /home/meissner/fsf-install-ppc64le/pu-binutils/bin/ld: ESMF_Clock.fppized.o:/home/meissner/spec-build/spec-2017-base-dev100-powerle9-vsx-mrmrpath-nomass-hugetlbfs-stdmalloc-shared-at12.0-64bit/benchspec/CPU/521.wrf_r/build/build_base_base_64.0000/ESMF_Clock.fppized.f90:223: more undefined references to `_gfortran_os_error_at' follow collect2: error: ld returned 1 exit status specmake: *** [diffwrf_521] Error 1 Here are the options used for compiling one benchark file: <...>/gfortran -c -o ESMF_Fraction.fppized.o -I. -I./netcdf/include \ -I./inc -g -mlittle -save-temps=obj -ffast-math -Ofast \ -mcpu=power9 -mrecip -funroll-loops \ -mno-pointers-to-nested-functions -msave-toc-indirect -m64 \ -I/opt/at12.0/include \ -I/opt/at12.0/lib/gcc/powerpc64le-linux-gnu/8.3.1/include \ -fstack-arrays -std=legacy -fconvert=big-endian \ -fno-range-check -fcray-pointer ESMF_Fraction.fppized.f90 -- Michael Meissner, IBM IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA email: meissner@linux.ibm.com, phone: +1 (978) 899-4797 ^ permalink raw reply [flat|nested] 8+ messages in thread
* [PATCHv2] Change the library search path when using --with-advance-toolchain 2019-10-03 23:32 ` Michael Meissner @ 2019-10-04 21:32 ` Tulio Magno Quites Machado Filho 2019-10-05 17:21 ` Segher Boessenkool ` (2 more replies) 0 siblings, 3 replies; 8+ messages in thread From: Tulio Magno Quites Machado Filho @ 2019-10-04 21:32 UTC (permalink / raw) To: Michael Meissner, gcc-patches; +Cc: segher Michael Meissner <meissner@linux.ibm.com> writes: > And then I built Spec 2006 and 2017 with my normal options on a power8 system > running Ubuntu and an older set of host libraries. If I have this patch > installed, it breaks linking some/all of the C++ and Fortran benchmarks, > because it finds the host C++/Fortran libraries before the AT 12.0 libraries. You have an usecase where the compiler has a different prefix. That requires an extra library directory, but it still has to come after the user lib directory. ---8<--- Remove all -L directories from LINK_OS_EXTRA_SPEC32 and LINK_OS_EXTRA_SPEC64 so that user directories specified at build time have higher preference over the advance toolchain libraries. Set MD_STARTFILE_PREFIX to $prefix/lib/ and MD_STARTFILE_PREFIX_1 to $at/lib/ so that a compiler library has preference over the Advance Toolchain libraries. 2019-10-04 Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com> * config.gcc: Move -L usage from LINK_OS_EXTRA_SPEC32 and LINK_OS_EXTRA_SPEC64 to MD_STARTFILE_PREFIX and MD_STARTFILE_PREFIX_1 when using --with-advance-toolchain. --- gcc/config.gcc | 11 ++++++----- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/gcc/config.gcc b/gcc/config.gcc index 481bc9586a7..511aeb454bf 100644 --- a/gcc/config.gcc +++ b/gcc/config.gcc @@ -4782,14 +4782,12 @@ case "${target}" in echo "#undef LINK_OS_EXTRA_SPEC32" echo "#define LINK_OS_EXTRA_SPEC32" \ "\"%(link_os_new_dtags)" \ - "-rpath $prefix/lib -rpath $at/lib" \ - "-L $prefix/lib -L $at/lib\"" + "-rpath $prefix/lib -rpath $at/lib\"" echo echo "#undef LINK_OS_EXTRA_SPEC64" echo "#define LINK_OS_EXTRA_SPEC64" \ "\"%(link_os_new_dtags)" \ - "-rpath $prefix/lib64 -rpath $at/lib64" \ - "-L $prefix/lib64 -L $at/lib64\"" + "-rpath $prefix/lib64 -rpath $at/lib64\"" echo echo "#undef LINK_OS_NEW_DTAGS_SPEC" echo "#define LINK_OS_NEW_DTAGS_SPEC" \ @@ -4802,7 +4800,10 @@ case "${target}" in echo "#define MD_EXEC_PREFIX \"$at/bin/\"" echo echo "#undef MD_STARTFILE_PREFIX" - echo "#define MD_STARTFILE_PREFIX \"$at/lib/\"") \ + echo "#define MD_STARTFILE_PREFIX \"$prefix/lib/\"" + echo + echo "#undef MD_STARTFILE_PREFIX_1" + echo "#define MD_STARTFILE_PREFIX_1 \"$at/lib/\"") \ > advance-toolchain.h else echo "Unknown advance-toolchain $with_advance_toolchain" -- 2.14.5 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCHv2] Change the library search path when using --with-advance-toolchain 2019-10-04 21:32 ` [PATCHv2] Change the library search path when using --with-advance-toolchain Tulio Magno Quites Machado Filho @ 2019-10-05 17:21 ` Segher Boessenkool 2019-10-23 21:51 ` Peter Bergner 2019-10-07 20:46 ` Michael Meissner 2019-10-08 15:10 ` Segher Boessenkool 2 siblings, 1 reply; 8+ messages in thread From: Segher Boessenkool @ 2019-10-05 17:21 UTC (permalink / raw) To: Tulio Magno Quites Machado Filho; +Cc: Michael Meissner, gcc-patches On Fri, Oct 04, 2019 at 06:31:34PM -0300, Tulio Magno Quites Machado Filho wrote: > Remove all -L directories from LINK_OS_EXTRA_SPEC32 and > LINK_OS_EXTRA_SPEC64 so that user directories specified at > build time have higher preference over the advance toolchain libraries. > > Set MD_STARTFILE_PREFIX to $prefix/lib/ and MD_STARTFILE_PREFIX_1 to > $at/lib/ so that a compiler library has preference over the Advance > Toolchain libraries. This is fine, approved for all branches. Thank you! And thanks to Mike for the testing. Segher > 2019-10-04 Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com> > > * config.gcc: Move -L usage from LINK_OS_EXTRA_SPEC32 and > LINK_OS_EXTRA_SPEC64 to MD_STARTFILE_PREFIX and > MD_STARTFILE_PREFIX_1 when using --with-advance-toolchain. ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCHv2] Change the library search path when using --with-advance-toolchain 2019-10-05 17:21 ` Segher Boessenkool @ 2019-10-23 21:51 ` Peter Bergner 0 siblings, 0 replies; 8+ messages in thread From: Peter Bergner @ 2019-10-23 21:51 UTC (permalink / raw) To: Segher Boessenkool, Tulio Magno Quites Machado Filho Cc: Michael Meissner, gcc-patches On 10/5/19 12:20 PM, Segher Boessenkool wrote: > On Fri, Oct 04, 2019 at 06:31:34PM -0300, Tulio Magno Quites Machado Filho wrote: >> Remove all -L directories from LINK_OS_EXTRA_SPEC32 and >> LINK_OS_EXTRA_SPEC64 so that user directories specified at >> build time have higher preference over the advance toolchain libraries. >> >> Set MD_STARTFILE_PREFIX to $prefix/lib/ and MD_STARTFILE_PREFIX_1 to >> $at/lib/ so that a compiler library has preference over the Advance >> Toolchain libraries. > > This is fine, approved for all branches. Thank you! And thanks to Mike > for the testing. I've committed the back ports to the FSF 7, 8 and 9 branches now after clean bootstraps and regtesting. Peter ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCHv2] Change the library search path when using --with-advance-toolchain 2019-10-04 21:32 ` [PATCHv2] Change the library search path when using --with-advance-toolchain Tulio Magno Quites Machado Filho 2019-10-05 17:21 ` Segher Boessenkool @ 2019-10-07 20:46 ` Michael Meissner 2019-10-08 15:10 ` Segher Boessenkool 2 siblings, 0 replies; 8+ messages in thread From: Michael Meissner @ 2019-10-07 20:46 UTC (permalink / raw) To: Tulio Magno Quites Machado Filho; +Cc: Michael Meissner, gcc-patches, segher On Fri, Oct 04, 2019 at 06:31:34PM -0300, Tulio Magno Quites Machado Filho wrote: > Michael Meissner <meissner@linux.ibm.com> writes: > > > And then I built Spec 2006 and 2017 with my normal options on a power8 system > > running Ubuntu and an older set of host libraries. If I have this patch > > installed, it breaks linking some/all of the C++ and Fortran benchmarks, > > because it finds the host C++/Fortran libraries before the AT 12.0 libraries. > > You have an usecase where the compiler has a different prefix. > That requires an extra library directory, but it still has to come after > the user lib directory. Once I fixed the perl script to remove the extra -L's from the spec config.cfg file that the perl script had put in for the previous version of --with-advance-toolchain=<xxx>, I was able to link one of the problematical benchmarks (2017 roms_r in this case). -- Michael Meissner, IBM IBM, M/S 2506R, 550 King Street, Littleton, MA 01460-6245, USA email: meissner@linux.ibm.com, phone: +1 (978) 899-4797 ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCHv2] Change the library search path when using --with-advance-toolchain 2019-10-04 21:32 ` [PATCHv2] Change the library search path when using --with-advance-toolchain Tulio Magno Quites Machado Filho 2019-10-05 17:21 ` Segher Boessenkool 2019-10-07 20:46 ` Michael Meissner @ 2019-10-08 15:10 ` Segher Boessenkool 2 siblings, 0 replies; 8+ messages in thread From: Segher Boessenkool @ 2019-10-08 15:10 UTC (permalink / raw) To: Tulio Magno Quites Machado Filho; +Cc: Michael Meissner, gcc-patches On Fri, Oct 04, 2019 at 06:31:34PM -0300, Tulio Magno Quites Machado Filho wrote: > Remove all -L directories from LINK_OS_EXTRA_SPEC32 and > LINK_OS_EXTRA_SPEC64 so that user directories specified at > build time have higher preference over the advance toolchain libraries. > > Set MD_STARTFILE_PREFIX to $prefix/lib/ and MD_STARTFILE_PREFIX_1 to > $at/lib/ so that a compiler library has preference over the Advance > Toolchain libraries. > > 2019-10-04 Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com> > > * config.gcc: Move -L usage from LINK_OS_EXTRA_SPEC32 and > LINK_OS_EXTRA_SPEC64 to MD_STARTFILE_PREFIX and > MD_STARTFILE_PREFIX_1 when using --with-advance-toolchain. Committed ( https://gcc.gnu.org/r276702 ). Thanks! Segher ^ permalink raw reply [flat|nested] 8+ messages in thread
* Re: [PATCH] Remove extra lib directory from --with-advance-toolchain 2019-10-03 17:08 [PATCH] Remove extra lib directory from --with-advance-toolchain Tulio Magno Quites Machado Filho 2019-10-03 23:32 ` Michael Meissner @ 2019-10-04 16:13 ` Jeff Law 1 sibling, 0 replies; 8+ messages in thread From: Jeff Law @ 2019-10-04 16:13 UTC (permalink / raw) To: Tulio Magno Quites Machado Filho, gcc-patches; +Cc: segher, meissner On 10/3/19 11:08 AM, Tulio Magno Quites Machado Filho wrote: > Remove the extra -L directory so that user directories specified at > build time have higher preference over the advance toolchain libraries. > > 2019-10-03 Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com> > > * config.gcc: Remove an extra -L from --with-advance-toolchain. Explicitly leaving this for the PPC maintainers. Y'all know what's best for the advance toolchain. Jeff ^ permalink raw reply [flat|nested] 8+ messages in thread
end of thread, other threads:[~2019-10-23 21:47 UTC | newest] Thread overview: 8+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2019-10-03 17:08 [PATCH] Remove extra lib directory from --with-advance-toolchain Tulio Magno Quites Machado Filho 2019-10-03 23:32 ` Michael Meissner 2019-10-04 21:32 ` [PATCHv2] Change the library search path when using --with-advance-toolchain Tulio Magno Quites Machado Filho 2019-10-05 17:21 ` Segher Boessenkool 2019-10-23 21:51 ` Peter Bergner 2019-10-07 20:46 ` Michael Meissner 2019-10-08 15:10 ` Segher Boessenkool 2019-10-04 16:13 ` [PATCH] Remove extra lib directory from --with-advance-toolchain Jeff Law
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).