[PATCH] D32200: [LV] Refactor ILV.vectorize[Loop]() by introducing LVP.executePlan()

[Matthew Simpson via Phabricator via llvm-commits]

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

Hey Ayal,

Sorry to lose track of this patch. It looks good to me.



================
Comment at: lib/Transforms/Vectorize/LoopVectorize.cpp:7880
     // If we decided that it is *legal* to vectorize the loop, then do it.
     InnerLoopVectorizer LB(L, PSE, LI, DT, TLI, TTI, AC, ORE, VF.Width, IC,
                            &LVL, &CM);
----------------
Do we know how this ended up begin "LB"? We might should change it to something more meaningful at some point (not in this patch, though).


https://reviews.llvm.org/D32200