[arch-commits] Commit in lib32-llvm/repos/multilib-x86_64 (9 files)
Evangelos Foutras
foutrelis at archlinux.org
Fri Jul 9 19:28:21 UTC 2021
Date: Friday, July 9, 2021 @ 19:28:21
Author: foutrelis
Revision: 976309
archrelease: copy trunk to multilib-x86_64
Added:
lib32-llvm/repos/multilib-x86_64/PKGBUILD
(from rev 976308, lib32-llvm/trunk/PKGBUILD)
lib32-llvm/repos/multilib-x86_64/add-fno-semantic-interposition.patch
(from rev 976308, lib32-llvm/trunk/add-fno-semantic-interposition.patch)
lib32-llvm/repos/multilib-x86_64/llvm-link-with-Bsymbolic-functions.patch
(from rev 976308, lib32-llvm/trunk/llvm-link-with-Bsymbolic-functions.patch)
lib32-llvm/repos/multilib-x86_64/no-strict-aliasing-DwarfCompileUnit.patch
(from rev 976308, lib32-llvm/trunk/no-strict-aliasing-DwarfCompileUnit.patch)
Deleted:
lib32-llvm/repos/multilib-x86_64/PKGBUILD
lib32-llvm/repos/multilib-x86_64/add-fno-semantic-interposition.patch
lib32-llvm/repos/multilib-x86_64/force-visibility-of-llvm-Any-to-external.patch
lib32-llvm/repos/multilib-x86_64/llvm-link-with-Bsymbolic-functions.patch
lib32-llvm/repos/multilib-x86_64/no-strict-aliasing-DwarfCompileUnit.patch
------------------------------------------------+
PKGBUILD | 235 +++++++++++------------
add-fno-semantic-interposition.patch | 44 ++--
force-visibility-of-llvm-Any-to-external.patch | 42 ----
llvm-link-with-Bsymbolic-functions.patch | 132 ++++++------
no-strict-aliasing-DwarfCompileUnit.patch | 26 +-
5 files changed, 216 insertions(+), 263 deletions(-)
Deleted: PKGBUILD
===================================================================
--- PKGBUILD 2021-07-09 19:28:13 UTC (rev 976308)
+++ PKGBUILD 2021-07-09 19:28:21 UTC (rev 976309)
@@ -1,120 +0,0 @@
-# Maintainer: Laurent Carlier <lordheavym at gmail.com>
-# Contributor: Evangelos Foutras <foutrelis at gmail.com>
-# Contributor: Jan "heftig" Steffens <jan.steffens at gmail.com>
-
-pkgname=('lib32-llvm' 'lib32-llvm-libs')
-pkgver=12.0.0
-pkgrel=2
-arch=('x86_64')
-url="https://llvm.org/"
-license=('custom:Apache 2.0 with LLVM Exception')
-makedepends=('cmake' 'ninja' 'lib32-libffi' 'lib32-zlib' 'python' 'gcc-multilib'
- 'lib32-libxml2')
-options=('staticlibs')
-_source_base=https://github.com/llvm/llvm-project/releases/download/llvmorg-$pkgver
-source=($_source_base/llvm-$pkgver.src.tar.xz{,.sig}
- force-visibility-of-llvm-Any-to-external.patch
- llvm-link-with-Bsymbolic-functions.patch
- add-fno-semantic-interposition.patch
- no-strict-aliasing-DwarfCompileUnit.patch)
-sha256sums=('49dc47c8697a1a0abd4ee51629a696d7bfe803662f2a7252a3b16fc75f3a8b50'
- 'SKIP'
- '98721af5a36af2a8e88c14a81b16d3929b12515d7d2d1ba385eb243dca3c32cb'
- '560ce1e206c19f4b86f4c583b743db0ad47a610418999350710aafd60ae50fcd'
- 'fc8c64267a5d179e9fc24fb2bc6150edef2598c83f5b2d138d14e05ce9f4e345'
- 'd1eff24508e35aae6c26a943dbaa3ef5acb60a145b008fd1ef9ac6f6c4faa662')
-validpgpkeys+=('B6C8F98282B944E3B0D5C2530FC3042E345AD05D') # Hans Wennborg <hans at chromium.org>
-validpgpkeys+=('474E22316ABF4785A88C6E8EA2C794A986419D8A') # Tom Stellard <tstellar at redhat.com>
-
-prepare() {
- cd "$srcdir/llvm-$pkgver.src"
- mkdir build
-
- # https://bugs.llvm.org/show_bug.cgi?id=48992
- patch -Np2 -i ../force-visibility-of-llvm-Any-to-external.patch
-
- # https://bugs.archlinux.org/task/70697
- patch -Np2 -i ../llvm-link-with-Bsymbolic-functions.patch
- # https://reviews.llvm.org/D102453
- patch -Np2 -i ../add-fno-semantic-interposition.patch
-
- # https://bugs.llvm.org/show_bug.cgi?id=50611#c3
- patch -Np2 -i ../no-strict-aliasing-DwarfCompileUnit.patch
-}
-
-build() {
- cd "$srcdir/llvm-$pkgver.src/build"
-
- export PKG_CONFIG_PATH="/usr/lib32/pkgconfig"
-
- cmake .. -G Ninja \
- -DCMAKE_BUILD_TYPE=Release \
- -DCMAKE_INSTALL_PREFIX=/usr \
- -DLLVM_LIBDIR_SUFFIX=32 \
- -DCMAKE_C_FLAGS:STRING=-m32 \
- -DCMAKE_CXX_FLAGS:STRING=-m32 \
- -DLLVM_TARGET_ARCH:STRING=i686 \
- -DLLVM_HOST_TRIPLE=$CHOST \
- -DLLVM_DEFAULT_TARGETS_TRIPLE="i686-pc-linux-gnu" \
- -DLLVM_BUILD_LLVM_DYLIB=ON \
- -DLLVM_LINK_LLVM_DYLIB=ON \
- -DLLVM_ENABLE_RTTI=ON \
- -DLLVM_ENABLE_FFI=ON \
- -DLLVM_BUILD_DOCS=OFF \
- -DLLVM_ENABLE_SPHINX=OFF \
- -DLLVM_ENABLE_DOXYGEN=OFF \
- -DLLVM_BINUTILS_INCDIR=/usr/include
-
- ninja all
-}
-
-package_lib32-llvm() {
- pkgdesc="Collection of modular and reusable compiler and toolchain technologies (32-bit)"
- depends=('lib32-llvm-libs' 'llvm')
-
- cd "$srcdir/llvm-$pkgver.src/build"
-
- DESTDIR="$pkgdir" ninja install
-
- # The runtime library goes into lib32-llvm-libs
- mv "$pkgdir"/usr/lib32/lib{LLVM,LTO,Remarks}*.so* "$srcdir"
- mv -f "$pkgdir"/usr/lib32/LLVMgold.so "$srcdir"
-
- # Fix permissions of static libs
- chmod -x "$pkgdir"/usr/lib32/*.a
-
- mv "$pkgdir/usr/bin/llvm-config" "$pkgdir/usr/lib32/llvm-config"
- mv "$pkgdir/usr/include/llvm/Config/llvm-config.h" \
- "$pkgdir/usr/lib32/llvm-config-32.h"
-
- rm -rf "$pkgdir"/usr/{bin,include,share/{doc,man,llvm,opt-viewer}}
-
- # Needed for multilib (https://bugs.archlinux.org/task/29951)
- # Header stub is taken from Fedora
- install -d "$pkgdir/usr/include/llvm/Config"
- mv "$pkgdir/usr/lib32/llvm-config-32.h" "$pkgdir/usr/include/llvm/Config/"
-
- mkdir "$pkgdir"/usr/bin
- mv "$pkgdir/usr/lib32/llvm-config" "$pkgdir/usr/bin/llvm-config32"
-
- install -Dm644 ../LICENSE.TXT "$pkgdir/usr/share/licenses/$pkgname/LICENSE"
-}
-
-package_lib32-llvm-libs() {
- pkgdesc="Low Level Virtual Machine (runtime library)(32-bit) "
- depends=('lib32-libffi' 'lib32-zlib' 'lib32-ncurses' 'lib32-libxml2' 'lib32-gcc-libs')
-
- install -d "$pkgdir/usr/lib32"
-
- cp -P \
- "$srcdir"/lib{LLVM,LTO,Remarks}*.so* \
- "$srcdir"/LLVMgold.so \
- "$pkgdir/usr/lib32/"
-
- # Symlink LLVMgold.so from /usr/lib/bfd-plugins
- # https://bugs.archlinux.org/task/28479
- install -d "$pkgdir/usr/lib32/bfd-plugins"
- ln -s ../LLVMgold.so "$pkgdir/usr/lib32/bfd-plugins/LLVMgold.so"
-
- install -Dm644 llvm-$pkgver.src/LICENSE.TXT "$pkgdir/usr/share/licenses/$pkgname/LICENSE"
-}
Copied: lib32-llvm/repos/multilib-x86_64/PKGBUILD (from rev 976308, lib32-llvm/trunk/PKGBUILD)
===================================================================
--- PKGBUILD (rev 0)
+++ PKGBUILD 2021-07-09 19:28:21 UTC (rev 976309)
@@ -0,0 +1,115 @@
+# Maintainer: Laurent Carlier <lordheavym at gmail.com>
+# Contributor: Evangelos Foutras <foutrelis at gmail.com>
+# Contributor: Jan "heftig" Steffens <jan.steffens at gmail.com>
+
+pkgname=('lib32-llvm' 'lib32-llvm-libs')
+pkgver=12.0.1
+pkgrel=1
+arch=('x86_64')
+url="https://llvm.org/"
+license=('custom:Apache 2.0 with LLVM Exception')
+makedepends=('cmake' 'ninja' 'lib32-libffi' 'lib32-zlib' 'python' 'gcc-multilib'
+ 'lib32-libxml2')
+options=('staticlibs')
+_source_base=https://github.com/llvm/llvm-project/releases/download/llvmorg-$pkgver
+source=($_source_base/llvm-$pkgver.src.tar.xz{,.sig}
+ llvm-link-with-Bsymbolic-functions.patch
+ add-fno-semantic-interposition.patch
+ no-strict-aliasing-DwarfCompileUnit.patch)
+sha256sums=('7d9a8405f557cefc5a21bf5672af73903b64749d9bc3a50322239f56f34ffddf'
+ 'SKIP'
+ '560ce1e206c19f4b86f4c583b743db0ad47a610418999350710aafd60ae50fcd'
+ 'fc8c64267a5d179e9fc24fb2bc6150edef2598c83f5b2d138d14e05ce9f4e345'
+ 'd1eff24508e35aae6c26a943dbaa3ef5acb60a145b008fd1ef9ac6f6c4faa662')
+validpgpkeys+=('B6C8F98282B944E3B0D5C2530FC3042E345AD05D') # Hans Wennborg <hans at chromium.org>
+validpgpkeys+=('474E22316ABF4785A88C6E8EA2C794A986419D8A') # Tom Stellard <tstellar at redhat.com>
+
+prepare() {
+ cd "$srcdir/llvm-$pkgver.src"
+ mkdir build
+
+ # https://bugs.archlinux.org/task/70697
+ patch -Np2 -i ../llvm-link-with-Bsymbolic-functions.patch
+ # https://reviews.llvm.org/D102453
+ patch -Np2 -i ../add-fno-semantic-interposition.patch
+
+ # https://bugs.llvm.org/show_bug.cgi?id=50611#c3
+ patch -Np2 -i ../no-strict-aliasing-DwarfCompileUnit.patch
+}
+
+build() {
+ cd "$srcdir/llvm-$pkgver.src/build"
+
+ export PKG_CONFIG_PATH="/usr/lib32/pkgconfig"
+
+ cmake .. -G Ninja \
+ -DCMAKE_BUILD_TYPE=Release \
+ -DCMAKE_INSTALL_PREFIX=/usr \
+ -DLLVM_LIBDIR_SUFFIX=32 \
+ -DCMAKE_C_FLAGS:STRING=-m32 \
+ -DCMAKE_CXX_FLAGS:STRING=-m32 \
+ -DLLVM_TARGET_ARCH:STRING=i686 \
+ -DLLVM_HOST_TRIPLE=$CHOST \
+ -DLLVM_DEFAULT_TARGETS_TRIPLE="i686-pc-linux-gnu" \
+ -DLLVM_BUILD_LLVM_DYLIB=ON \
+ -DLLVM_LINK_LLVM_DYLIB=ON \
+ -DLLVM_ENABLE_RTTI=ON \
+ -DLLVM_ENABLE_FFI=ON \
+ -DLLVM_BUILD_DOCS=OFF \
+ -DLLVM_ENABLE_SPHINX=OFF \
+ -DLLVM_ENABLE_DOXYGEN=OFF \
+ -DLLVM_BINUTILS_INCDIR=/usr/include
+
+ ninja all
+}
+
+package_lib32-llvm() {
+ pkgdesc="Collection of modular and reusable compiler and toolchain technologies (32-bit)"
+ depends=('lib32-llvm-libs' 'llvm')
+
+ cd "$srcdir/llvm-$pkgver.src/build"
+
+ DESTDIR="$pkgdir" ninja install
+
+ # The runtime library goes into lib32-llvm-libs
+ mv "$pkgdir"/usr/lib32/lib{LLVM,LTO,Remarks}*.so* "$srcdir"
+ mv -f "$pkgdir"/usr/lib32/LLVMgold.so "$srcdir"
+
+ # Fix permissions of static libs
+ chmod -x "$pkgdir"/usr/lib32/*.a
+
+ mv "$pkgdir/usr/bin/llvm-config" "$pkgdir/usr/lib32/llvm-config"
+ mv "$pkgdir/usr/include/llvm/Config/llvm-config.h" \
+ "$pkgdir/usr/lib32/llvm-config-32.h"
+
+ rm -rf "$pkgdir"/usr/{bin,include,share/{doc,man,llvm,opt-viewer}}
+
+ # Needed for multilib (https://bugs.archlinux.org/task/29951)
+ # Header stub is taken from Fedora
+ install -d "$pkgdir/usr/include/llvm/Config"
+ mv "$pkgdir/usr/lib32/llvm-config-32.h" "$pkgdir/usr/include/llvm/Config/"
+
+ mkdir "$pkgdir"/usr/bin
+ mv "$pkgdir/usr/lib32/llvm-config" "$pkgdir/usr/bin/llvm-config32"
+
+ install -Dm644 ../LICENSE.TXT "$pkgdir/usr/share/licenses/$pkgname/LICENSE"
+}
+
+package_lib32-llvm-libs() {
+ pkgdesc="Low Level Virtual Machine (runtime library)(32-bit) "
+ depends=('lib32-libffi' 'lib32-zlib' 'lib32-ncurses' 'lib32-libxml2' 'lib32-gcc-libs')
+
+ install -d "$pkgdir/usr/lib32"
+
+ cp -P \
+ "$srcdir"/lib{LLVM,LTO,Remarks}*.so* \
+ "$srcdir"/LLVMgold.so \
+ "$pkgdir/usr/lib32/"
+
+ # Symlink LLVMgold.so from /usr/lib/bfd-plugins
+ # https://bugs.archlinux.org/task/28479
+ install -d "$pkgdir/usr/lib32/bfd-plugins"
+ ln -s ../LLVMgold.so "$pkgdir/usr/lib32/bfd-plugins/LLVMgold.so"
+
+ install -Dm644 llvm-$pkgver.src/LICENSE.TXT "$pkgdir/usr/share/licenses/$pkgname/LICENSE"
+}
Deleted: add-fno-semantic-interposition.patch
===================================================================
--- add-fno-semantic-interposition.patch 2021-07-09 19:28:13 UTC (rev 976308)
+++ add-fno-semantic-interposition.patch 2021-07-09 19:28:21 UTC (rev 976309)
@@ -1,22 +0,0 @@
-diff --git a/llvm/cmake/modules/HandleLLVMOptions.cmake b/llvm/cmake/modules/HandleLLVMOptions.cmake
---- a/llvm/cmake/modules/HandleLLVMOptions.cmake
-+++ b/llvm/cmake/modules/HandleLLVMOptions.cmake
-@@ -305,6 +305,17 @@
- # On Windows all code is PIC. MinGW warns if -fPIC is used.
- else()
- add_flag_or_print_warning("-fPIC" FPIC)
-+ # Enable interprocedural optimizations for non-inline functions which would
-+ # otherwise be disabled due to GCC -fPIC's default.
-+ #
-+ # Note: Clang allows IPO for -fPIC so this optimization is less effective.
-+ # Older Clang may support -fno-semantic-interposition but it used local
-+ # aliases to optimize global variables, which is incompatible with copy
-+ # relocations due to -fno-pic.
-+ if (CMAKE_COMPILER_IS_GNUCXX OR (CMAKE_CXX_COMPILER_ID MATCHES "Clang" AND
-+ CMAKE_CXX_COMPILER_VERSION GREATER_EQUAL 13))
-+ add_flag_if_supported("-fno-semantic-interposition" FNO_SEMANTIC_INTERPOSITION)
-+ endif()
- endif()
- # GCC for MIPS can miscompile LLVM due to PR37701.
- if(CMAKE_COMPILER_IS_GNUCXX AND LLVM_NATIVE_ARCH STREQUAL "Mips" AND
-
Copied: lib32-llvm/repos/multilib-x86_64/add-fno-semantic-interposition.patch (from rev 976308, lib32-llvm/trunk/add-fno-semantic-interposition.patch)
===================================================================
--- add-fno-semantic-interposition.patch (rev 0)
+++ add-fno-semantic-interposition.patch 2021-07-09 19:28:21 UTC (rev 976309)
@@ -0,0 +1,22 @@
+diff --git a/llvm/cmake/modules/HandleLLVMOptions.cmake b/llvm/cmake/modules/HandleLLVMOptions.cmake
+--- a/llvm/cmake/modules/HandleLLVMOptions.cmake
++++ b/llvm/cmake/modules/HandleLLVMOptions.cmake
+@@ -305,6 +305,17 @@
+ # On Windows all code is PIC. MinGW warns if -fPIC is used.
+ else()
+ add_flag_or_print_warning("-fPIC" FPIC)
++ # Enable interprocedural optimizations for non-inline functions which would
++ # otherwise be disabled due to GCC -fPIC's default.
++ #
++ # Note: Clang allows IPO for -fPIC so this optimization is less effective.
++ # Older Clang may support -fno-semantic-interposition but it used local
++ # aliases to optimize global variables, which is incompatible with copy
++ # relocations due to -fno-pic.
++ if (CMAKE_COMPILER_IS_GNUCXX OR (CMAKE_CXX_COMPILER_ID MATCHES "Clang" AND
++ CMAKE_CXX_COMPILER_VERSION GREATER_EQUAL 13))
++ add_flag_if_supported("-fno-semantic-interposition" FNO_SEMANTIC_INTERPOSITION)
++ endif()
+ endif()
+ # GCC for MIPS can miscompile LLVM due to PR37701.
+ if(CMAKE_COMPILER_IS_GNUCXX AND LLVM_NATIVE_ARCH STREQUAL "Mips" AND
+
Deleted: force-visibility-of-llvm-Any-to-external.patch
===================================================================
--- force-visibility-of-llvm-Any-to-external.patch 2021-07-09 19:28:13 UTC (rev 976308)
+++ force-visibility-of-llvm-Any-to-external.patch 2021-07-09 19:28:21 UTC (rev 976309)
@@ -1,42 +0,0 @@
-From 3d3abc22b3ef189813a3b9061c2a90ba86a32f44 Mon Sep 17 00:00:00 2001
-From: serge-sans-paille <sguelton at redhat.com>
-Date: Tue, 11 May 2021 20:46:58 +0200
-Subject: [PATCH] Force visibility of llvm::Any to external
-
-llvm::Any::TypeId::Id relies on the uniqueness of the address of a static
-variable defined in a template function. hidden visibility implies vague linkage
-for that variable, which does not guarantee the uniqueness of the address across
-a binary and a shared library. This totally breaks the implementation of
-llvm::Any.
-
-Ideally, setting visibility to llvm::Any::TypeId::Id should be enough,
-unfortunately this doesn't work as expected and we lack time (before 12.0.1
-release) to understand why setting the visibility to llvm::Any does work.
-
-See https://gcc.gnu.org/wiki/Visibility and
-https://gcc.gnu.org/onlinedocs/gcc/Vague-Linkage.html
-for more information on that topic.
-
-Differential Revision: https://reviews.llvm.org/D101972
----
- llvm/include/llvm/ADT/Any.h | 7 ++++++-
- 1 file changed, 6 insertions(+), 1 deletion(-)
-
-diff --git a/llvm/include/llvm/ADT/Any.h b/llvm/include/llvm/ADT/Any.h
-index 15b114f7932d..e513586845a1 100644
---- a/llvm/include/llvm/ADT/Any.h
-+++ b/llvm/include/llvm/ADT/Any.h
-@@ -23,7 +23,12 @@
-
- namespace llvm {
-
--class Any {
-+class LLVM_EXTERNAL_VISIBILITY Any {
-+
-+ // The `Typeid<T>::Id` static data member below is a globally unique
-+ // identifier for the type `T`. It is explicitly marked with default
-+ // visibility so that when `-fvisibility=hidden` is used, the loader still
-+ // merges duplicate definitions across DSO boundaries.
- template <typename T> struct TypeId { static const char Id; };
-
- struct StorageBase {
Deleted: llvm-link-with-Bsymbolic-functions.patch
===================================================================
--- llvm-link-with-Bsymbolic-functions.patch 2021-07-09 19:28:13 UTC (rev 976308)
+++ llvm-link-with-Bsymbolic-functions.patch 2021-07-09 19:28:21 UTC (rev 976309)
@@ -1,66 +0,0 @@
-From 4f05f4c8e66bc76b1d94f5283494404382e3bacd Mon Sep 17 00:00:00 2001
-From: Fangrui Song <i at maskray.me>
-Date: Thu, 13 May 2021 13:44:57 -0700
-Subject: [PATCH] [CMake][ELF] Link libLLVM.so and libclang-cpp.so with
- -Bsymbolic-functions
-
-llvm-dev message: https://lists.llvm.org/pipermail/llvm-dev/2021-May/150465.html
-
-In an ELF shared object, a default visibility defined symbol is preemptible by
-default. This creates some missed optimization opportunities.
--Bsymbolic-functions is more aggressive than our current -fvisibility-inlines-hidden
-(present since 2012) as it applies to all function definitions. It can
-
-* avoid PLT for cross-TU function calls && reduce dynamic symbol lookup
-* reduce dynamic symbol lookup for taking function addresses and optimize out GOT/TOC on x86-64/ppc64
-
-In a -DLLVM_TARGETS_TO_BUILD=X86 build, the number of JUMP_SLOT decreases from 12716 to 1628, and the number of GLOB_DAT decreases from 1918 to 1313
-The built clang with `-DLLVM_LINK_LLVM_DYLIB=on -DCLANG_LINK_CLANG_DYLIB=on` is significantly faster.
-See the Linux kernel build result https://bugs.archlinux.org/task/70697
-
-Note: the performance of -fno-semantic-interposition -Bsymbolic-functions
-libLLVM.so and libclang-cpp.so is close to a PIE binary linking against
-`libLLVM*.a` and `libclang*.a`. When the host compiler is Clang,
--Bsymbolic-functions is the major contributor. On x86-64 (with GOTPCRELX) and
-ppc64 ELFv2, the GOT/TOC relocations can be optimized.
-
-Some implication:
-
-Interposing a subset of functions is no longer supported.
-(This is fragile on ELF and unsupported on Mach-O at all. For Mach-O we don't
-use `ld -interpose` or `-flat_namespace`)
-
-Compiling a program which takes the address of any LLVM function with
-`{gcc,clang} -fno-pic` and expects the address to equal to the address taken
-from libLLVM.so or libclang-cpp.so is unsupported. I am fairly confident that
-llvm-project shouldn't have different behaviors depending on such pointer
-equality (as we've been using -fvisibility-inlines-hidden which applies to
-inline functions for a long time), but if we accidentally do, users should be
-aware that they should not make assumption on pointer equality in `-fno-pic`
-mode.
-
-See more on https://maskray.me/blog/2021-05-09-fno-semantic-interposition
-
-Reviewed By: phosek
-
-Differential Revision: https://reviews.llvm.org/D102090
----
- llvm/tools/llvm-shlib/CMakeLists.txt | 5 +++++
- 1 file changed, 5 insertions(+)
-
-diff --git a/llvm/tools/llvm-shlib/CMakeLists.txt b/llvm/tools/llvm-shlib/CMakeLists.txt
-index b0ee19049e6f..03e1383ec8b4 100644
---- a/llvm/tools/llvm-shlib/CMakeLists.txt
-+++ b/llvm/tools/llvm-shlib/CMakeLists.txt
-@@ -50,6 +50,11 @@ if(LLVM_BUILD_LLVM_DYLIB)
- # Solaris ld does not accept global: *; so there is no way to version *all* global symbols
- set(LIB_NAMES -Wl,--version-script,${LLVM_LIBRARY_DIR}/tools/llvm-shlib/simple_version_script.map ${LIB_NAMES})
- endif()
-+ # Optimize function calls for default visibility definitions to avoid PLT and
-+ # reduce dynamic relocations.
-+ # Note: for -fno-pic default, the address of a function may be different from
-+ # inside and outside libLLVM.so.
-+ target_link_options(LLVM PRIVATE LINKER:-Bsymbolic-functions)
- elseif("${CMAKE_SYSTEM_NAME}" STREQUAL "Darwin")
- set(LIB_NAMES -Wl,-all_load ${LIB_NAMES})
- endif()
Copied: lib32-llvm/repos/multilib-x86_64/llvm-link-with-Bsymbolic-functions.patch (from rev 976308, lib32-llvm/trunk/llvm-link-with-Bsymbolic-functions.patch)
===================================================================
--- llvm-link-with-Bsymbolic-functions.patch (rev 0)
+++ llvm-link-with-Bsymbolic-functions.patch 2021-07-09 19:28:21 UTC (rev 976309)
@@ -0,0 +1,66 @@
+From 4f05f4c8e66bc76b1d94f5283494404382e3bacd Mon Sep 17 00:00:00 2001
+From: Fangrui Song <i at maskray.me>
+Date: Thu, 13 May 2021 13:44:57 -0700
+Subject: [PATCH] [CMake][ELF] Link libLLVM.so and libclang-cpp.so with
+ -Bsymbolic-functions
+
+llvm-dev message: https://lists.llvm.org/pipermail/llvm-dev/2021-May/150465.html
+
+In an ELF shared object, a default visibility defined symbol is preemptible by
+default. This creates some missed optimization opportunities.
+-Bsymbolic-functions is more aggressive than our current -fvisibility-inlines-hidden
+(present since 2012) as it applies to all function definitions. It can
+
+* avoid PLT for cross-TU function calls && reduce dynamic symbol lookup
+* reduce dynamic symbol lookup for taking function addresses and optimize out GOT/TOC on x86-64/ppc64
+
+In a -DLLVM_TARGETS_TO_BUILD=X86 build, the number of JUMP_SLOT decreases from 12716 to 1628, and the number of GLOB_DAT decreases from 1918 to 1313
+The built clang with `-DLLVM_LINK_LLVM_DYLIB=on -DCLANG_LINK_CLANG_DYLIB=on` is significantly faster.
+See the Linux kernel build result https://bugs.archlinux.org/task/70697
+
+Note: the performance of -fno-semantic-interposition -Bsymbolic-functions
+libLLVM.so and libclang-cpp.so is close to a PIE binary linking against
+`libLLVM*.a` and `libclang*.a`. When the host compiler is Clang,
+-Bsymbolic-functions is the major contributor. On x86-64 (with GOTPCRELX) and
+ppc64 ELFv2, the GOT/TOC relocations can be optimized.
+
+Some implication:
+
+Interposing a subset of functions is no longer supported.
+(This is fragile on ELF and unsupported on Mach-O at all. For Mach-O we don't
+use `ld -interpose` or `-flat_namespace`)
+
+Compiling a program which takes the address of any LLVM function with
+`{gcc,clang} -fno-pic` and expects the address to equal to the address taken
+from libLLVM.so or libclang-cpp.so is unsupported. I am fairly confident that
+llvm-project shouldn't have different behaviors depending on such pointer
+equality (as we've been using -fvisibility-inlines-hidden which applies to
+inline functions for a long time), but if we accidentally do, users should be
+aware that they should not make assumption on pointer equality in `-fno-pic`
+mode.
+
+See more on https://maskray.me/blog/2021-05-09-fno-semantic-interposition
+
+Reviewed By: phosek
+
+Differential Revision: https://reviews.llvm.org/D102090
+---
+ llvm/tools/llvm-shlib/CMakeLists.txt | 5 +++++
+ 1 file changed, 5 insertions(+)
+
+diff --git a/llvm/tools/llvm-shlib/CMakeLists.txt b/llvm/tools/llvm-shlib/CMakeLists.txt
+index b0ee19049e6f..03e1383ec8b4 100644
+--- a/llvm/tools/llvm-shlib/CMakeLists.txt
++++ b/llvm/tools/llvm-shlib/CMakeLists.txt
+@@ -50,6 +50,11 @@ if(LLVM_BUILD_LLVM_DYLIB)
+ # Solaris ld does not accept global: *; so there is no way to version *all* global symbols
+ set(LIB_NAMES -Wl,--version-script,${LLVM_LIBRARY_DIR}/tools/llvm-shlib/simple_version_script.map ${LIB_NAMES})
+ endif()
++ # Optimize function calls for default visibility definitions to avoid PLT and
++ # reduce dynamic relocations.
++ # Note: for -fno-pic default, the address of a function may be different from
++ # inside and outside libLLVM.so.
++ target_link_options(LLVM PRIVATE LINKER:-Bsymbolic-functions)
+ elseif("${CMAKE_SYSTEM_NAME}" STREQUAL "Darwin")
+ set(LIB_NAMES -Wl,-all_load ${LIB_NAMES})
+ endif()
Deleted: no-strict-aliasing-DwarfCompileUnit.patch
===================================================================
--- no-strict-aliasing-DwarfCompileUnit.patch 2021-07-09 19:28:13 UTC (rev 976308)
+++ no-strict-aliasing-DwarfCompileUnit.patch 2021-07-09 19:28:21 UTC (rev 976309)
@@ -1,13 +0,0 @@
-diff --git a/llvm/lib/CodeGen/AsmPrinter/CMakeLists.txt b/llvm/lib/CodeGen/AsmPrinter/CMakeLists.txt
-index eb924282a75e..85929b54d6ce 100644
---- a/llvm/lib/CodeGen/AsmPrinter/CMakeLists.txt
-+++ b/llvm/lib/CodeGen/AsmPrinter/CMakeLists.txt
-@@ -44,3 +44,8 @@ add_llvm_component_library(LLVMAsmPrinter
- Support
- Target
- )
-+
-+# https://bugs.llvm.org/show_bug.cgi?id=50611#c3
-+if (CMAKE_COMPILER_IS_GNUCXX)
-+ set_source_files_properties(DwarfCompileUnit.cpp PROPERTIES COMPILE_FLAGS -fno-strict-aliasing)
-+endif()
Copied: lib32-llvm/repos/multilib-x86_64/no-strict-aliasing-DwarfCompileUnit.patch (from rev 976308, lib32-llvm/trunk/no-strict-aliasing-DwarfCompileUnit.patch)
===================================================================
--- no-strict-aliasing-DwarfCompileUnit.patch (rev 0)
+++ no-strict-aliasing-DwarfCompileUnit.patch 2021-07-09 19:28:21 UTC (rev 976309)
@@ -0,0 +1,13 @@
+diff --git a/llvm/lib/CodeGen/AsmPrinter/CMakeLists.txt b/llvm/lib/CodeGen/AsmPrinter/CMakeLists.txt
+index eb924282a75e..85929b54d6ce 100644
+--- a/llvm/lib/CodeGen/AsmPrinter/CMakeLists.txt
++++ b/llvm/lib/CodeGen/AsmPrinter/CMakeLists.txt
+@@ -44,3 +44,8 @@ add_llvm_component_library(LLVMAsmPrinter
+ Support
+ Target
+ )
++
++# https://bugs.llvm.org/show_bug.cgi?id=50611#c3
++if (CMAKE_COMPILER_IS_GNUCXX)
++ set_source_files_properties(DwarfCompileUnit.cpp PROPERTIES COMPILE_FLAGS -fno-strict-aliasing)
++endif()
More information about the arch-commits
mailing list