[PATCH] D59723: [NewPassManager] Adding pass tuning options: loop vectorize.

[Alina Sbirlea via Phabricator via llvm-commits]

asbirlea updated this revision to Diff 194188.
asbirlea added a comment.

Attempting to merge defaults to a single cl::opt.
Please note the comments on defaults in lib/Transforms/Vectorize/LoopVectorize.cpp. Feedback there very welcome.

Details:
The default for LoopVectorize in the PassManagerBuilder is false. (opt checks for this, and sets LoopVectorize to true based on Olevel/size)
The default when creating the LoopVectorize with no arguments, is true (VectorizeOnlyWhenForced = false).


Repository:
  rL LLVM

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

https://reviews.llvm.org/D59723

Files:
  include/llvm/Passes/PassBuilder.h
  include/llvm/Transforms/Vectorize.h
  include/llvm/Transforms/Vectorize/LoopVectorize.h
  lib/LTO/LTOBackend.cpp
  lib/Passes/PassBuilder.cpp
  lib/Transforms/IPO/PassManagerBuilder.cpp
  lib/Transforms/Vectorize/LoopVectorize.cpp
  tools/opt/NewPMDriver.cpp
  unittests/IR/PassBuilderCallbacksTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D59723.194188.patch
Type: text/x-patch
Size: 8418 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190408/6418e349/attachment.bin>