[PATCH] D100979: [loop-idiom][NFC] Extract processLoopStoreOfLoopLoad into a helper function

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 22 14:00:01 PDT 2021


hoy added inline comments.


================
Comment at: llvm/lib/Transforms/Scalar/LoopIdiomRecognize.cpp:1088
   const SCEVAddRecExpr *LoadEv =
       cast<SCEVAddRecExpr>(SE->getSCEV(LI->getPointerOperand()));
+  Value *LoadPtr = LI->getPointerOperand();
----------------
zhuhan0 wrote:
> hoy wrote:
> > Nit: reuse LoadPtr below?
> Could you clarify? Where should I reuse `LoadPtr`?
There is a use of LI->getPointerOperand() at line 1088 that can be replaced by LoadPtr. Sorry for the confusion.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100979



More information about the llvm-commits mailing list