[PATCH] D72386: [ThinLTO] pass UnrollLoops/VectorizeLoop/VectorizeSLP in CGOpts down to pass builder in ltobackend
Teresa Johnson via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 8 08:37:37 PST 2020
tejohnson added a comment.
Thinking through this some more - this patch fixes the distributed ThinLTO backend case, but not the in-process ThinLTO case. For that, both lld and gold plugin will need to set this up. And since they don't have the clang options, they will just need to set to some reasonable default, probably either hardwired on like in the old PM or set based on the CGOptLevel they set from linker options. However, they both default to -O2 unless a special plugin or lld linker option is set, so we'd probably want to enable these passes there at O2 <https://reviews.llvm.org/owners/package/2/>+.
Repository:
rL LLVM
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D72386/new/
https://reviews.llvm.org/D72386
More information about the llvm-commits
mailing list