[PATCH] D14138: Fix two issues in MergeConsecutiveStores:

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Sun Nov 1 23:45:36 PST 2015


arsenm added a subscriber: arsenm.

================
Comment at: lib/CodeGen/SelectionDAG/DAGCombiner.cpp:11172
@@ +11171,3 @@
+    // operations that potentially overlap past each-other.
+    if (Index->getAlignment()*8 < MemVT.getSizeInBits())
+      break;
----------------
This should use getStoreSize instead of * 8


http://reviews.llvm.org/D14138





More information about the llvm-commits mailing list