[PATCH] D10477: Expand loop reroll to handle loop with multiple induction variables and negative increment -part 1/3

Lawrence Hu lawrence at codeaurora.org
Wed Jul 15 10:39:28 PDT 2015


hulx2000 added inline comments.

================
Comment at: lib/Transforms/Scalar/LoopRerollPass.cpp:677
@@ -661,2 +676,3 @@
 
-  for (auto *UU : BO->users()) {
+  if (!BO && !isa<GetElementPtrInst>(U))
+    return false;
----------------
jmolloy wrote:
> I'm not sure how adding GEP support here relates to negative values. Was this an accidentally added hunk?
Oh, sorry, this is left over when I split my patch, it is require for pointer induction variable though, but I can remove it for now.

Will fix the rest, thanks.``


Repository:
  rL LLVM

http://reviews.llvm.org/D10477







More information about the llvm-commits mailing list