[PATCH] D135808: [LoopInterchange] Correcting the profitability checking for vectorization

[Ramkrishnan Narayanan Komala via Phabricator via llvm-commits]

ram-NK created this revision.
ram-NK added reviewers: fhahn, congzhe, Meinersbur.
Herald added a subscriber: hiraditya.
Herald added a project: All.
ram-NK requested review of this revision.
Herald added subscribers: llvm-commits, pcwang-thead.
Herald added a project: LLVM.

If inner loop is loop independent then, loop interchange is profitable. If outer loop is loop independent then, loop interchange will not have added benefit. Corrected the dependency checking inside isProfitableForVectorization(). This change will add more profitable cases in loop interchange.

Failed test case corrected.

  In pr43797-lcssa-for-multiple-outer-loop-blocks.ll, loop interchange is not profitable for all loops in the file.


https://reviews.llvm.org/D135808

Files:
  llvm/lib/Transforms/Scalar/LoopInterchange.cpp
  llvm/test/Transforms/LoopInterchange/pr43797-lcssa-for-multiple-outer-loop-blocks.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D135808.467235.patch
Type: text/x-patch
Size: 6803 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20221012/3f45d136/attachment.bin>