diff options
author | Stephen Sachs <stephenmsachs@gmail.com> | 2022-02-15 18:47:29 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2022-02-15 17:47:29 +0000 |
commit | 79f22423b81c2c269ebc191698348c58d08f3a0f (patch) | |
tree | dbc6f6a84ac5c4ba0aec4310e560f4851eaa259e /lib | |
parent | cebe4fdf1d7cc3a150c5ee5080104e58dc7557c5 (diff) | |
download | spack-79f22423b81c2c269ebc191698348c58d08f3a0f.tar.gz spack-79f22423b81c2c269ebc191698348c58d08f3a0f.tar.bz2 spack-79f22423b81c2c269ebc191698348c58d08f3a0f.tar.xz spack-79f22423b81c2c269ebc191698348c58d08f3a0f.zip |
intel compiler: fix link time error with `LLVMgold.so` (#28731)
The Intel compiler will, at link time, call `ld -plugin LLVMgold.so`, which
expects libraries like `libimfo.so` to be found either in the `LD_LIBRARY_PATH` or
in `LLVMgold.so`s RPATH.
As `LLVMgold.so` already uses RUNPATH, I used that to extend this to the
necessary library locations.
This PR should fix issues:
https://github.com/spack/spack/issues/10308
https://github.com/spack/spack/issues/18606
https://github.com/spack/spack/issues/17100
https://github.com/spack/spack/issues/21237
https://github.com/spack/spack/issues/4261
Co-authored-by: Stephen Sachs <stesachs@amazon.com>
Diffstat (limited to 'lib')
-rw-r--r-- | lib/spack/spack/build_systems/intel.py | 38 |
1 files changed, 38 insertions, 0 deletions
diff --git a/lib/spack/spack/build_systems/intel.py b/lib/spack/spack/build_systems/intel.py index 1cf7923fb4..6e4a2a970e 100644 --- a/lib/spack/spack/build_systems/intel.py +++ b/lib/spack/spack/build_systems/intel.py @@ -24,6 +24,7 @@ from llnl.util.filesystem import ( install, ) +import spack.error from spack.build_environment import dso_suffix from spack.package import InstallError, PackageBase, run_after from spack.util.environment import EnvironmentModifications @@ -1333,5 +1334,42 @@ class IntelPackage(PackageBase): debug_print(os.getcwd()) return + @property + def base_lib_dir(self): + """Provide the library directory located in the base of Intel installation. + """ + d = self.normalize_path('') + d = os.path.join(d, 'lib') + + debug_print(d) + return d + + @run_after('install') + def modify_LLVMgold_rpath(self): + """Add libimf.so and other required libraries to the RUNPATH of LLVMgold.so. + + These are needed explicitly at dependent link time when + `ld -plugin LLVMgold.so` is called by the compiler. + """ + if self._has_compilers: + LLVMgold_libs = find_libraries('LLVMgold', self.base_lib_dir, + shared=True, recursive=True) + # Ignore ia32 entries as they mostly ignore throughout the rest + # of the file. + # The first entry in rpath preserves the original, the seconds entry + # is the location of libimf.so. If this relative location is changed + # in compiler releases, then we need to search for libimf.so instead + # of this static path. + for lib in LLVMgold_libs: + if not self.spec.satisfies('^patchelf'): + raise spack.error.SpackError( + 'Attempting to patch RPATH in LLVMgold.so.' + + '`patchelf` dependency should be set in package.py' + ) + patchelf = Executable('patchelf') + rpath = ':'.join([patchelf('--print-rpath', lib, output=str).strip(), + '$ORIGIN/../compiler/lib/intel64_lin']) + patchelf('--set-rpath', rpath, lib) + # Check that self.prefix is there after installation run_after('install')(PackageBase.sanity_check_prefix) |