[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