[PATCH] D143069: [DAGCombine] Allow DAGCombine to remove dead masked stores

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:11121-11124
+        (!MST->getMemoryVT().isScalableVector() ||
+         MST1->getMemoryVT().isScalableVector()) &&
+        MST->getMemoryVT().getStoreSize().getKnownMinValue() ==
+            MST1->getMemoryVT().getStoreSize().getKnownMinValue()) {
----------------
Can this be replaced by:

  ElementCount::isKnownLE(MST1->getMemoryVT().getStoreSize(),
                          MST->getMemoryVT().getStoreSize())

?


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

https://reviews.llvm.org/D143069