[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 a comment.

@congzhe  and @bmahjour  comments are addressed.



================
Comment at: llvm/lib/Transforms/Scalar/LoopInterchange.cpp:1166
+  // analyze the loopnest (e.g., due to delinearization issues).
+  if (CostMap.find(InnerLoop) == CostMap.end() || CostMap.find(OuterLoop) != CostMap.end() ||
+     (CC && CC->getLoopCost(*InnerLoop) == CC->getLoopCost(*OuterLoop))) {
----------------
congzhe wrote:
> 
Corrected the condition. any of the loop is failed to determine the loop nest and equal locality then only checks the profit of vectorization.


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

https://reviews.llvm.org/D135808