When refactoring vectorization flags, vectorization was disabled by default in the new pass manager.
This patch re-enables is for both managers, and changes the assumptions opt makes, based on the new defaults.
The patch also removes the -disable-loop-vectorization flag and leaves a single way to force *disable* vectorization for -O[23s] (via -vectorize-loops=false). We can add a cl::opt to force enabling under -O[01z] if anyone actually needs this.
Comments in opt.cpp should also help clarify the intended use of the flags to enable/disable vectorization.
"specific user" doesn't make sense here to me.
Not sure why it makes sense for DisableLoopVectorization to be true by default...
I'm not super worried about the existing behavior of cl::opt flags. Those aren't stable. So I'd suggest picking names and set of such flags that are simple and easy to explain.