[PATCH] D45552: [NFC][LV][LoopUtil] Move LoopVectorizationLegality to Analysis tree

[Renato Golin via Phabricator via llvm-commits]

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

Right, pure code movement, and a good clean up at that, LGTM.

Just one nit in the comments and can go as soon as its deps are approved. Thanks!



================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:22
 // 2. LoopVectorizationLegality - A unit that checks for the legality
-//    of the vectorization.
+//    of the vectorization. [Moved out to Analysis]
 // 3. InnerLoopVectorizer - A unit that performs the actual
----------------
I wouldn't say "moved out", but just reference the file like "in LoopVectorizationLegality".


https://reviews.llvm.org/D45552