Remove an optimization that isn't.

llvm-svn: 3419

1 files changed, 7 insertions, 5 deletions

diff --git a/llvm/lib/Transforms/Scalar/DecomposeMultiDimRefs.cpp b/llvm/lib/Transforms/Scalar/DecomposeMultiDimRefs.cpp
index 657640471a1..7bfa329b57c 100644
--- a/llvm/lib/Transforms/Scalar/DecomposeMultiDimRefs.cpp
+++ b/llvm/lib/Transforms/Scalar/DecomposeMultiDimRefs.cpp
@@ -88,13 +88,15 @@ IsZero(Value* idx)
 bool
 DecomposePass::decomposeArrayRef(BasicBlock::iterator &BBI)
 {
+  // FIXME: If condition below
   MemAccessInst &MAI = cast<MemAccessInst>(*BBI);
+  // FIXME: If condition below
 
-  // If this instr two or fewer arguments and the first argument is 0,
-  // the decomposed version is identical to the instruction itself.
-  // This is common enough that it is worth checking for explicitly...
-  if (MAI.getNumIndices() == 0 ||
-      (MAI.getNumIndices() <= 2 && IsZero(*MAI.idx_begin()))) {
+  // If this instr has no indexes, then the decomposed version is identical to
+  // the instruction itself.  FIXME: this should go away once GEP is the only
+  // MAI
+  //
+  if (MAI.getNumIndices() == 0) {
     ++BBI;
     return false;
   }