This is an archive of the discontinued LLVM Phabricator instance.

gold-plugin: Do not set codegen opt level based on LTO opt level.
ClosedPublic

Authored by pcc on Feb 7 2018, 1:07 PM.

Details

Summary

The LTO opt level should not affect the codegen opt level, and indeed
it does not affect it in lld. Ideally the codegen opt level should
be controlled by an IR-level attribute based on the compile-time opt
level, but that hasn't been implemented yet.

Event Timeline

pcc created this revision.Feb 7 2018, 1:07 PM
This revision was not accepted when it landed; it landed in state Needs Review.Feb 7 2018, 6:45 PM
This revision was automatically updated to reflect the committed changes.
ychen added a subscriber: ychen.Dec 24 2019, 9:47 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 24 2019, 9:47 PM
Herald added a subscriber: dexonsmith. · View Herald Transcript

Ideally the codegen opt level should be controlled by an IR-level attribute based on the compile-time opt level, but that hasn't been implemented yet.

Why isn't it a good default in the meantime?
What will the default be?

(never mind this is an old revision, it just happen to popup in my inbox because of an unrelated update to it)