diff options
| author | andymwood <andy.m.wood@gmail.com> | 2021-01-05 21:11:29 +0000 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2021-01-05 13:11:29 -0800 |
| commit | 35d81a9006a5fa32a012a40874ac6cbdefd4a259 (patch) | |
| tree | e9ec873a08f009b71e2aca28cc0bf9b0343955bd | |
| parent | 61c1b71d38e62a5af81b3b7b8a8d12b954d99f0a (diff) | |
| download | spack-35d81a9006a5fa32a012a40874ac6cbdefd4a259.tar.gz spack-35d81a9006a5fa32a012a40874ac6cbdefd4a259.tar.bz2 spack-35d81a9006a5fa32a012a40874ac6cbdefd4a259.tar.xz spack-35d81a9006a5fa32a012a40874ac6cbdefd4a259.zip | |
VTK: add -no-ipo for builds using intel compiler (#20480)
Co-authored-by: Adam J. Stewart <ajstewart426@gmail.com>
| -rw-r--r-- | var/spack/repos/builtin/packages/vtk/package.py | 19 |
1 files changed, 15 insertions, 4 deletions
diff --git a/var/spack/repos/builtin/packages/vtk/package.py b/var/spack/repos/builtin/packages/vtk/package.py index 834a96b7ab..d7a7126830 100644 --- a/var/spack/repos/builtin/packages/vtk/package.py +++ b/var/spack/repos/builtin/packages/vtk/package.py @@ -265,11 +265,10 @@ class Vtk(CMakePackage): '-DVTK_USE_X:BOOL=ON', '-DVTK_USE_COCOA:BOOL=OFF']) + compile_flags = [] + if spec.satisfies('@:6.1.0'): - cmake_args.extend([ - '-DCMAKE_C_FLAGS=-DGLX_GLXEXT_LEGACY', - '-DCMAKE_CXX_FLAGS=-DGLX_GLXEXT_LEGACY' - ]) + compile_flags.append('-DGLX_GLXEXT_LEGACY') # VTK 6.1.0 (and possibly earlier) does not use # NETCDF_CXX_ROOT to detect NetCDF C++ bindings, so @@ -298,4 +297,16 @@ class Vtk(CMakePackage): cmake_args.append( '-DVTK_MODULE_ENABLE_VTK_IOMotionFX:BOOL=OFF') + # -no-ipo prevents an internal compiler error from multi-file + # optimization (https://github.com/spack/spack/issues/20471) + if '%intel' in spec: + compile_flags.append('-no-ipo') + + if compile_flags: + compile_flags = ' '.join(compile_flags) + cmake_args.extend([ + '-DCMAKE_C_FLAGS={0}'.format(compile_flags), + '-DCMAKE_CXX_FLAGS={0}'.format(compile_flags) + ]) + return cmake_args |