TrkGlobalChi2Fitter: Optimize some functions, even in dbg builds.

The function GlobalChi2Fitter::calculateJac makes heavy use of Eigen and runs extremely slowly in dbg builds. Adjust to compile this function with optimization, even in dbg builds. For the MuonCombinedConfig unit test, this results in a 5% speedup in the dbg build.

WARNING: big files (>100K) are found in the changeset

296K in file Tracking/TrkFitter/TrkGlobalChi2Fitter/src/GlobalChi2Fitter.cxx

This merge request affects 1 package:

• Tracking/TrkFitter/TrkGlobalChi2Fitter

This merge request affects 2 files:

• Tracking/TrkFitter/TrkGlobalChi2Fitter/CMakeLists.txt
• Tracking/TrkFitter/TrkGlobalChi2Fitter/src/GlobalChi2Fitter.cxx

Adding @amorley as watcher

added Tracking master review-pending-level-1 labels

CI Result SUCCESS (hash cccad87a)

	Athena	AthSimulation	AthGeneration	AnalysisBase	AthAnalysis	DetCommon
externals
cmake
make
required tests
optional tests

Full details available on this CI monitor view
Athena: number of compilation errors 0, warnings 0
AthSimulation: number of compilation errors 0, warnings 0
AthGeneration: number of compilation errors 0, warnings 0
AnalysisBase: number of compilation errors 0, warnings 0
AthAnalysis: number of compilation errors 0, warnings 0
DetCommon: number of compilation errors 0, warnings 0
For experts only: Jenkins output [CI-MERGE-REQUEST-CC7 31128]

Hi, I'm passing this MR to L2 to take a look at the changes made in CMakeLists mainly. Also I'm adding @akraszna as a watcher...

Cheers, Francisco --L1

added review-pending-level-2 label and removed review-pending-level-1 label

The changes seems fine to me, optimising one function in dbg build. Approving.

Pavol [as L2 MR shifter]

added review-approved label and removed review-pending-level-2 label

mentioned in commit 2442d411

merged

added sweep:ignore label