[PATCH] D28868: [LV] Run loop-simplify and LCSSA explicitly instead of "requiring" them

Michael Kuperstein via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 18 11:55:40 PST 2017


mkuper created this revision.
Herald added a subscriber: mzolotukhin.

This changes the vectorizer to explicitly use the loopsimplify and lcssa utils, instead of "requiring" the transformations as if they were analyses.

This is not NFC, since it changes the LCSSA behavior - we no longer run LCSSA for all loops, but rather only for the loops we expect to modify. It doesn't look like we depend on LCSSA anywhere in legality/profitability analysis, except what was just fixed in https://reviews.llvm.org/rL292424.


https://reviews.llvm.org/D28868

Files:
  lib/Transforms/Vectorize/LoopVectorize.cpp
  test/Transforms/LoopVectorize/partial-lcssa.ll
  test/Transforms/LoopVectorize/pr31190.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28868.84872.patch
Type: text/x-patch
Size: 6807 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170118/92373c4b/attachment.bin>


More information about the llvm-commits mailing list