[PATCH] D19202: [PR27390] [CodeGen] Reject indexed loads in CombinerDAG.

[Marcin Kościelnicki via llvm-commits]

koriakin added inline comments.

================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAG.cpp:6964
@@ -6963,1 +6963,3 @@
     return false;
+  if (LD->isIndexed() || Base->isIndexed())
+    return false;
----------------
hfinkel wrote:
> Okay. Please check in DAGCombiner::CombineConsecutiveLoads instead. Otherwise, LGTM.
Hmm, I'm not sure the logic in areNonVolatileConsecutiveLoads works right in the presence of indexed load anyhow (in particular, pre-indexing needs to be taken into account).  I'll take a look.


Repository:
  rL LLVM

http://reviews.llvm.org/D19202