[PATCH] D96600: [TTI] Unify FavorPostInc and FavorBackedgeIndex into getAddressingMode

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 12 06:16:11 PST 2021


samparker added a comment.

Shouldn't this be an NFC..? Looks like we should be favouring post-incs for MVE, even at optsize.



================
Comment at: llvm/lib/Transforms/Scalar/LoopStrengthReduce.cpp:1265
 
-      if (TTI->shouldFavorPostInc()) {
+      if (AMK == TTI::AMK_PostIndexed) {
         const SCEV *LoopStep = AR->getStepRecurrence(*SE);
----------------
else if


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

https://reviews.llvm.org/D96600



More information about the llvm-commits mailing list