[Lldb-commits] [PATCH] D65482: [DAGCombiner] Add an option to control whether or not to enable store merging

Wei Mi via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Tue Jul 30 15:38:26 PDT 2019

wmi created this revision.
wmi added a reviewer: tejohnson.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.

Add an option to control whether or not to enable store merging in dag combiner so we can workaround some bug more easily.




Index: lib/CodeGen/SelectionDAG/DAGCombiner.cpp
--- lib/CodeGen/SelectionDAG/DAGCombiner.cpp
+++ lib/CodeGen/SelectionDAG/DAGCombiner.cpp
@@ -111,6 +111,11 @@
   MaySplitLoadIndex("combiner-split-load-index", cl::Hidden, cl::init(true),
                     cl::desc("DAG combiner may split indexing from loads"));
+static cl::opt<bool>
+    EnableStoreMerging("combiner-store-merging", cl::Hidden, cl::init(true),
+                       cl::desc("DAG combiner enable merging multiple stores "
+                                "into a wider store"));
 static cl::opt<unsigned> TokenFactorInlineLimit(
     "combiner-tokenfactor-inline-limit", cl::Hidden, cl::init(2048),
     cl::desc("Limit the number of operands to inline for Token Factors"));
@@ -15521,7 +15526,7 @@
 bool DAGCombiner::MergeConsecutiveStores(StoreSDNode *St) {
-  if (OptLevel == CodeGenOpt::None)
+  if (OptLevel == CodeGenOpt::None || !EnableStoreMerging)
     return false;
   EVT MemVT = St->getMemoryVT();

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D65482.212456.patch
Type: text/x-patch
Size: 1062 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20190730/34c77557/attachment.bin>

More information about the lldb-commits mailing list