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

[Ramkrishnan Narayanan Komala via Phabricator via llvm-commits]

ram-NK marked 2 inline comments as done.
ram-NK added inline comments.


================
Comment at: llvm/test/Transforms/LoopInterchange/pr43797-lcssa-for-multiple-outer-loop-blocks.ll:11
 
+; CHECK:      Name:            InterchangeNotProfitable
+; CHECK-NEXT: Function:        test1
----------------
bmahjour wrote:
> I'm worried about losing functional coverage by avoiding interchange here due to profitability. Can you play with `-loop-interchange-threshold` or make slight changes to the memory accesses to make the test case profitable?
These changes are not needed after correcting the dependency check in loop interchange.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135808/new/

https://reviews.llvm.org/D135808