[PATCH] D34273: [SCEV] Use depth limit instead of local cache for SExt and ZExt

Wei Mi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 28 16:13:29 PDT 2017


wmi added inline comments.


================
Comment at: lib/Analysis/ScalarEvolution.cpp:1430
 
-  const SCEV *PreStart = getPreStartForExtend<ExtendOpTy>(AR, Ty, SE, Cache);
+  const SCEV *PreStart = getPreStartForExtend<ExtendOpTy>(AR, Ty, SE);
   if (!PreStart)
----------------
Maybe also pass Depth to getPreStartForExtend? we have SE->*GetExtendExpr() called inside of getPreStartForExtend. 


https://reviews.llvm.org/D34273





More information about the llvm-commits mailing list