Page MenuHomePhabricator

[CMake] Use -O0 for unittests under full LTO as well
ClosedPublic

Authored by phosek on May 18 2021, 1:10 PM.

Details

Summary

We already use -O0 for unittests under ThinLTO, do the same for full LTO
where the compile time costs to runtime benefits tradeoff is even worse.

Diff Detail

Event Timeline

phosek created this revision.May 18 2021, 1:10 PM
phosek requested review of this revision.May 18 2021, 1:10 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 18 2021, 1:10 PM
smeenai accepted this revision.May 18 2021, 1:25 PM

LGTM, makes a lot of sense.

This revision is now accepted and ready to land.May 18 2021, 1:25 PM
This revision was landed with ongoing or failed builds.May 18 2021, 1:41 PM
This revision was automatically updated to reflect the committed changes.