[PATCH] D116895: Fix a missed opportunity to optimize consecutive stores.

Hongtao Yu via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 10 00:35:52 PST 2022


hoy added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:17497
+      if (I.getOperandNo() == 0 && isa<StoreSDNode>(*I)) {
+        TryToAddCandidate(I);
+      }
----------------
Does it matter if Store3 interferes with one of the loads? Wondering if it is dealt with somewhere, otherwise merging store3 with the stores following the loads might cause be a problem.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116895/new/

https://reviews.llvm.org/D116895



More information about the llvm-commits mailing list