Merge pull request #8244 from cms-sw/lto-branch-chg

sync lto changes and enable lto for lto branch

compilation_flags.file

@@ -1,9 +1,7 @@
-%define lto_build_flags -flto -fipa-icf -flto-odr-type-merging -fno-fat-lto-objects -Wodr
+#Special architecture related flags
 %if "%{?arch_build_flags:set}" != "set"
 %ifarch ppc64le
 %define arch_build_flags  -mcpu=power8 -mtune=power8 --param=l1-cache-size=64 --param=l1-cache-line-size=128 --param=l2-cache-size=512
-%undefine lto_build_flags
-%define lto_build_flags %{nil}
 %endif
 %ifarch aarch64
 %define arch_build_flags -march=armv8-a -mno-outline-atomics

compilation_flags_lto.file

@@ -0,0 +1,13 @@
+#Enable/disable LTO builds
+%define enable_lto 1
+
+#LTO related flags
+%ifarch ppc64le
+%undefine enable_lto
+%define enable_lto 0
+%endif
+%if "%{enable_lto}" == "1"
+%define lto_build_flags -flto -fipa-icf -flto-odr-type-merging -fno-fat-lto-objects -Wodr
+%else
+%define lto_build_flags %{nil}
+%endif

dd4hep.spec

@@ -1,5 +1,6 @@
 ### RPM external dd4hep v01-23x
 ## INCLUDE compilation_flags
+## INCLUDE compilation_flags_lto
 
 %define tag 5c3b494f047ee025b2e32303c16ad854bfbb342d
 %define branch master
@@ -10,50 +11,45 @@ Source: git+https://github.com/%{github_user}/DD4hep.git?obj=%{branch}/%{tag}&ex
 BuildRequires: cmake
 Requires: root boost clhep xerces-c geant4
 
-%prep
-
-%setup -n %{n}-%{realversion}
-
-%build
 %if "%{?arch_build_flags}"
 %define build_flags -fPIC %{arch_build_flags} %{lto_build_flags}
 %else
 %define build_flags -fPIC %{lto_build_flags}
 %endif
 
+%define cmake_fixed_args \\\
+  -DCMAKE_INSTALL_PREFIX='%{i}' \\\
+  -DCMAKE_CXX_FLAGS="%{build_flags}" \\\
+  -DCMAKE_STATIC_LIBRARY_CXX_FLAGS="%{build_flags}" \\\
+  -DCMAKE_STATIC_LIBRARY_C_FLAGS="%{build_flags}" \\\
+  -DBoost_NO_BOOST_CMAKE=ON \\\
+  -DDD4HEP_USE_XERCESC=ON \\\
+  -DDD4HEP_USE_PYROOT=ON \\\
+  -DCMAKE_AR=$(which gcc-ar) \\\
+  -DCMAKE_RANLIB=$(which gcc-ranlib) \\\
+  -DCMAKE_CXX_STANDARD=17 \\\
+  -DCMAKE_BUILD_TYPE=Release \\\
+  -DDD4HEP_USE_GEANT4_UNITS=ON \\\
+  -DXERCESC_ROOT_DIR=${XERCES_C_ROOT} \\\
+  -DCMAKE_PREFIX_PATH="${CLHEP_ROOT};${XERCES_C_ROOT}"
+
+%prep
+
+%setup -n %{n}-%{realversion}
+
+%build
+
 export BOOST_ROOT
-CMAKE_ARGS="-DCMAKE_INSTALL_PREFIX='%{i}' \
-      -DBoost_NO_BOOST_CMAKE=ON \
-      -DDD4HEP_USE_XERCESC=ON \
-      -DXERCESC_ROOT_DIR=${XERCES_C_ROOT} \
-      -DDD4HEP_USE_PYROOT=ON \
-      -DCMAKE_AR=$(which gcc-ar) \
-      -DCMAKE_RANLIB=$(which gcc-ranlib) \
-      -DCMAKE_CXX_STANDARD=17 \
-      -DCMAKE_BUILD_TYPE=Release \
-      -DDD4HEP_USE_GEANT4_UNITS=ON \
-      -DCMAKE_PREFIX_PATH=${CLHEP_ROOT};${XERCES_C_ROOT}"
 
 #Build normal Shared D4Hep without Geant4
 rm -rf ../build; mkdir ../build; cd ../build
-cmake $CMAKE_ARGS \
-      -DBUILD_SHARED_LIBS=ON \
-      -DCMAKE_CXX_FLAGS="%{build_flags}" \
-      -DCMAKE_STATIC_LIBRARY_CXX_FLAGS="%{build_flags}" \
-      -DCMAKE_STATIC_LIBRARY_C_FLAGS="%{build_flags}" \
-      ../%{n}-%{realversion}
+cmake %{cmake_fixed_args} -DBUILD_SHARED_LIBS=ON ../%{n}-%{realversion}
 make %{makeprocesses} VERBOSE=1
 make install
 
 #Building DDG4 static
 rm -rf ../build-g4; mkdir ../build-g4; cd ../build-g4
-cmake $CMAKE_ARGS \
-      -DBUILD_SHARED_LIBS=OFF \
-      -DDD4HEP_USE_GEANT4=ON  \
-      -DCMAKE_CXX_FLAGS="%{build_flags}" \
-      -DCMAKE_STATIC_LIBRARY_CXX_FLAGS="%{build_flags}" \
-      -DCMAKE_STATIC_LIBRARY_C_FLAGS="%{build_flags}" \
-      ../%{n}-%{realversion}
+cmake %{cmake_fixed_args} -DBUILD_SHARED_LIBS=OFF -DDD4HEP_USE_GEANT4=ON ../%{n}-%{realversion}
 cd DDG4
 make %{makeprocesses} VERBOSE=1
 for lib in $(ls ../lib/libDDG4*.a | sed 's|.a$||'); do

geant4.spec

@@ -1,5 +1,6 @@
 ### RPM external geant4 10.7.2
 ## INCLUDE compilation_flags
+## INCLUDE compilation_flags_lto
 %define use_vecgeom 1
 %define tag %{realversion}
 %define branch geant4-10.7-release

scram-project-build.file

@@ -1,5 +1,6 @@
 ### FILE scram-project-build
 ## NO_AUTO_DEPENDENCY
+## INCLUDE compilation_flags_lto
 # FIXME: support building all platforms together like scram does?
 # FIXME: automatic sub-packages for "doc" etc?
 %define cmssw_libs biglib/%{cmsplatf} lib/%{cmsplatf}
@@ -53,7 +54,7 @@ BuildRequires: dwz
 %endif
 
 %if "%{?configtag:set}" != "set"
-%define configtag       V07-03-11
+%define configtag       V07-03-12
 %endif
 
 %if "%{?cvssrc:set}" != "set"
@@ -116,7 +117,9 @@ rm -rf config %{srctree} poison
 
 cd %_builddir
 echo %{configtag} > %_builddir/config/config_tag
-%_builddir/config/updateConfig.py -p %{ucprojtype} -v %v -s $SCRAMV1_VERSION -t ${%{toolconf}} --keys SCRAM_COMPILER=%{scram_compiler} \
+%_builddir/config/updateConfig.py -p %{ucprojtype} -v %v -s $SCRAMV1_VERSION -t ${%{toolconf}} \
+  --keys SCRAM_COMPILER=%{scram_compiler} \
+  --keys ENABLE_LTO=%{enable_lto} \
 %if "%{?gitcommit:set}" == "set"
   --keys PROJECT_GIT_HASH=%{gitcommit}
 %else

scram-tools.file/tool-env.file

@@ -1,4 +1,5 @@
 ## INCLUDE compilation_flags
+## INCLUDE compilation_flags_lto
 ## INCLUDE cuda-flags
 
 export ROOT_CXXMODULES="0"
@@ -16,6 +17,8 @@ export COMPILER_CXXFLAGS="%{arch_build_flags}"
 export ORACLE_ENV_ROOT=""
 export CUDA_FLAGS="%{nvcc_cuda_flags}"
 export CUDA_HOST_CXXFLAGS="%{nvcc_stdcxx}"
+%if "%{?lto_build_flags}"
 export LTO_FLAGS="%{lto_build_flags}"
+%endif
 if [ $(echo ' %requiredtools ' | grep ' python3 ' |wc -l) -gt 0 ] ; then export PYTHON3_LIB_SITE_PACKAGES ; fi
 if [ $(echo ' %requiredtools ' | grep ' python ' |wc -l) -gt 0 ] ;  then export PYTHON_LIB_SITE_PACKAGES ; fi

vecgeom.spec

@@ -1,5 +1,6 @@
 ### RPM external vecgeom v1.1.17
 ## INCLUDE compilation_flags
+## INCLUDE compilation_flags_lto
 %define tag ed9a40412c354652262ec80af449f5531206e52c
 Source: git+https://gitlab.cern.ch/VecGeom/VecGeom.git?obj=master/%{tag}&export=%{n}-%{realversion}&output=/%{n}-%{realversion}.tgz
 BuildRequires: cmake gmake