Re: [PATCH] ltmain.in: Pass through -fopenmp=*

Ileana Dumitrescu writes: > Thank you! This patch has been merged in. Thank you! > > On 24/03/2024 10:26, Sam James wrote: >> Clang uses -fopenmp=* to allow choosing between libgomp and libomp. >> * build-aux/ltmain.in: Pass through -fopenmp=*. >> Signed-off-by: S

[PATCH] ltmain.in: Pass through -fopenmp=*

Clang uses -fopenmp=* to allow choosing between libgomp and libomp. * build-aux/ltmain.in: Pass through -fopenmp=*. Signed-off-by: Sam James --- build-aux/ltmain.in | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/build-aux/ltmain.in b/build-aux/ltmain.in index af4970be

Re: [PATCH 2/3] ltmain.in: Parse additional clang options

Richard Purdie writes: > clang uses -rtlib and --unwindlib to select proper compiler runtime in > some cases. Therefore pass these options to linker when found in > ldflags Thanks, I was planning on sending this one too. > > * build-aux/ltmain.in: Handle clang linker options > --- >

[PATCH] .gitmodules: use HTTPS for cloning gnulib

--- .gitmodules | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/.gitmodules b/.gitmodules index e561a15b..fc4bd926 100644 --- a/.gitmodules +++ b/.gitmodules @@ -1,6 +1,6 @@ [submodule "gnulib"] path = gnulib -url = git://git.savannah.gnu.org/gnulib.git +

[PATCH] ltmain.in: Pass through -fdiagnostics-color, -frecord-gcc-switches

* -fdiagnostics-color: who doesn't like color? No reason to strip it. This makes the link phase stick out like a sore thumb if there's warnings emitted. * -frecord-gcc-switches: we use it in Gentoo to check if a package respects *FLAGS. Note that we do not need to include

[PATCH] Allow -Werror and -Werror=* through flag filtering

* build-aux/ltmain.in (func_mode_link): allow -Werror and -Werror=* through flat filtering at link time. This is needed for detecting likely-runtime problems with LTO using e.g. -Werror=odr or -Werror=lto-type-mismatch. Bug: https://bugs.gentoo.org/884767 Signed-off-by: Sam James

Re: [PATCH v2 00/12] Rebased version of Yocto patches

> On 16 Apr 2022, at 18:58, Sam James wrote: > > Done after discussion w/ Alex (thanks!). Rebased on master after > 2.4.7 release. Ping. Could we maybe get the easy ones in and then revisit the bits which received comments (unclear to me what we need to do for those)? s

Re: [PATCH v2 01/12] ltmain.sh: Fix sysroot paths being encoded into RPATHs

> On 17 Apr 2022, at 05:55, Alex Ameen wrote: > > This was all green down the line on the test suite on multiple systems ( > don't get too excited yet ) until I found bugs in the testsuite... > > I see how this flew under the radar previously though - currently there are > no tests which

[PATCH v2 12/12] libtool: Fix support for NIOS2 processor

From: Marek Vasut The name of the system contains the string "nios2". This string is caught by the some of the greedy checks for OS/2 in libtool, in particular the *os2* branches of switch statements match for the nios2 string, which results in incorrect behavior of libtool. This patch adds an

[PATCH v2 09/12] libtool.m4: Handle "/" as a sysroot correctly

From: Richard Purdie Update libtool.m4 to resolve a problem with lt_sysroot not being properly updated if the option '--with[-libtool]-sysroot' is not provided when running the 'configure' script for a package so that "/" as a sysroot is handled correctly by libtool. Signed-off-by: Richard

[PATCH v2 10/12] ltmain.in: Handle prefix-map compiler options correctly

From: Richard Purdie If lto is enabled, we need the prefix-map variables to be passed to the linker. Add these to the list of options libtool passes through. Signed-off-by: Richard Purdie --- build-aux/ltmain.in | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git

[PATCH v2 06/12] Makefile.am: make sure autoheader run before autoconf

From: Mingli Yu autoheader will update ../libtool-2.4.6/libltdl/config-h.in which autoconf needs, so there comes a race sometimes as below: | configure.ac:45: error: required file 'config-h.in' not found | touch '../libtool-2.4.6/libltdl/config-h.in' So make sure autoheader run before

[PATCH v2 04/12] ltmain.in: Don't encode RATHS which match default linker paths

From: Richard Purdie We don't want to add RPATHS which match default linker search paths, they're a waste of space. This patch filters libtools list of paths to encoode and removes the ones we don't need. Libtool may be passed link paths of the form "/usr/lib/../lib" so normalize the paths

[PATCH v2 11/12] Makefile.am: make sure autoheader run before automake

From: Mingli Yu When use automake to generate Makefile.in from Makefile.am, there comes below race: | configure.ac:45: error: required file 'config-h.in' not found It is because the file config-h.in in updating process by autoheader, so make automake run after autoheader to avoid the above

[PATCH v2 07/12] libtool: Check for static libs for internal compiler libraries

From: Khem Raj Libtool checks only for libraries linked as -l* when trying to find internal compiler libraries. Clang, however uses the absolute path to link its internal libraries e.g. compiler_rt. This patch handles clang's statically linked libraries when finding internal compiler libraries.

[PATCH v2 08/12] ltmain.in: Add missing sysroot to library path

From: Khem Raj When using a sysroot we should append it to libdir, which is helpful in cross builds as the system is staged in the sysroot. For normal builds, i.e. when lt_sysroot is not set, it will still behave the same and add -L/usr/lib to the relink command. Signed-off-by: Richard Purdie

[PATCH v2 01/12] ltmain.sh: Fix sysroot paths being encoded into RPATHs

From: Richard Purdie There is a bug where RPATHs could end up containing sysroot values when cross compiling which is obviously incorrect. Strip out sysroot components from libdir when building RPATH values to avoid this. Signed-off-by: Richard Purdie --- build-aux/ltmain.in | 14

[PATCH v2 03/12] libtool.m4: For reproducibility stop encoding hostname in libtool script

From: Richard Purdie For reproducibilty, stop encoding the hostname into the libtool script, this isn't really adding much to debugging and most distros are carrying such a patch now as reproducibility is important. Signed-off-by: Richard Purdie --- m4/libtool.m4 | 1 - 1 file changed, 1

[PATCH v2 05/12] libtool.m4: Rename the --with-sysroot option to avoid conflict with gcc/binutils

From: Khem Raj This patch renames the --with-sysroot option to --with-libtool-sysroot to avoid namespace conflict with binutils, gcc and other toolchain components since these componets also add that option to configure and this becomes confusing and conflicting otherwise. Signed-off-by:

[PATCH v2 02/12] ltmain.in: Handle trailing slashes on install commands correctly

From: Richard Purdie A command like: libtool --mode=install /usr/bin/install -c gck-roots-store-standalone.la '/image/usr/lib/gnome-keyring/standalone/' where the path ends with a trailing slash currently fails. This occurs in software like gnome-keyring or pulseaudio and is because the

[PATCH v2 00/12] Rebased version of Yocto patches

Done after discussion w/ Alex (thanks!). Rebased on master after 2.4.7 release. Khem Raj (3): libtool.m4: Rename the --with-sysroot option to avoid conflict with gcc/binutils libtool: Check for static libs for internal compiler libraries ltmain.in: Add missing sysroot to library path