[PATCH] D72386: [ThinLTO] pass UnrollLoops/VectorizeLoop/VectorizeSLP in CGOpts down to pass builder in ltobackend

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added inline comments.
Herald added a subscriber: StephenFan.


================
Comment at: lld/ELF/LTO.cpp:96
 
+  c.PTO.LoopVectorization = c.OptLevel > 1;
+  c.PTO.SLPVectorization = c.OptLevel > 1;
----------------
xbolva00 wrote:
> COFF?
This came up recently on discourse: https://discourse.llvm.org/t/unintended-lto-configuration-differences-between-elf-and-coff/62636

This patch is a couple years old now and @wmi isn't working actively on LLVM anymore, maybe someone working on COFF who can test it there can send a patch to make this consistent?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72386/new/

https://reviews.llvm.org/D72386