[PATCH] D11051: Swap loop invariant GEP with loop variant GEP to allow more LICM.

[Quentin Colombet via llvm-commits]

qcolombet added a comment.

Hi Lawrence,

There are no diffs on the test-suite, so you’re good on that side :).

I don’t have any problems with the patch and let you address Andy’s comment.

Thanks,
-Quentin


================
Comment at: lib/Transforms/Scalar/SeparateConstOffsetFromGEP.cpp:1092
@@ +1091,3 @@
+    GetElementPtrInst *FirstGEP, GetElementPtrInst *SecondGEP, Loop *CurLoop) {
+  if (!FirstGEP || FirstGEP->getNumUses() != 1)
+    return false;
----------------
hasOneUse


Repository:
  rL LLVM

http://reviews.llvm.org/D11051