[PATCH] D61091: Enable LoopVectorization by default.

Wed Apr 24 17:21:19 PDT 2019

chandlerc accepted this revision.
chandlerc added a comment.
This revision is now accepted and ready to land.

I like the patch, LGTM.

Slight correction to your email for posterity:

In D61091#1477975 <https://reviews.llvm.org/D61091#1477975>, @asbirlea wrote:

> Remove `DisableLoopVectorization` flag.
>  This leaves a single way to force-enable vectorization for O1 <https://reviews.llvm.org/owners/package/1/>/Oz (via -loop-vectorize).

I think you mean (and your patch does), a single way to force *disable* vectorization for -O[23s] (via -vectorize-loops=false).

And we can add a `cl::opt` to force enabling under -O[01z] if anyone actually needs this.

Repository:
  rL LLVM

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

https://reviews.llvm.org/D61091