[PATCH] D102096: [DAGCombiner] Fix DAG combine store elimination, different address space.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 10 10:03:00 PDT 2021


arsenm requested changes to this revision.
arsenm added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:17935
+          ST->getMemoryVT() == ST1->getMemoryVT() &&
+          ST->getPointerInfo().getAddrSpace() ==
+              ST1->getPointerInfo().getAddrSpace()) {
----------------
You don't need the getPointerInfo, StoreSDNode directly provides getAddressSpace


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

https://reviews.llvm.org/D102096



More information about the llvm-commits mailing list