[PATCH] D22295: Introduce GlobalSplit pass.

[Eli Friedman via llvm-commits]

eli.friedman added inline comments.

================
Comment at: lib/Transforms/IPO/GlobalSplit.cpp:51
@@ +50,3 @@
+        !isa<ConstantInt>(GEP->getOperand(2)))
+      return false;
+  }
----------------
Don't you also need to check the users of the GEP?  The result of getInBoundsIndex() only has a semantic effect for loads and stores.


https://reviews.llvm.org/D22295