Skip to content

Commit b216ca5

Browse files
author
Hal Finkel
committed
[NVPTX] Remove MemIntrinsicSDNode/MemSDNode duplicate checking
As of r214452, isa<MemSDNode> will return true for nodes for which isa<MemIntrinsicSDNode> will return true (classof now respects the actual class hierarchy). So we no longer need to check for both MemIntrinsicSDNode and MemSDNode separately. No functionality change intended. llvm-svn: 215523
1 parent a0de40a commit b216ca5

File tree

1 file changed

+0
-7
lines changed

1 file changed

+0
-7
lines changed

llvm/lib/Target/NVPTX/NVPTXISelDAGToDAG.cpp

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -5041,17 +5041,10 @@ bool NVPTXDAGToDAGISel::SelectADDRri64(SDNode *OpNode, SDValue Addr,
50415041
bool NVPTXDAGToDAGISel::ChkMemSDNodeAddressSpace(SDNode *N,
50425042
unsigned int spN) const {
50435043
const Value *Src = nullptr;
5044-
// Even though MemIntrinsicSDNode is a subclas of MemSDNode,
5045-
// the classof() for MemSDNode does not include MemIntrinsicSDNode
5046-
// (See SelectionDAGNodes.h). So we need to check for both.
50475044
if (MemSDNode *mN = dyn_cast<MemSDNode>(N)) {
50485045
if (spN == 0 && mN->getMemOperand()->getPseudoValue())
50495046
return true;
50505047
Src = mN->getMemOperand()->getValue();
5051-
} else if (MemSDNode *mN = dyn_cast<MemIntrinsicSDNode>(N)) {
5052-
if (spN == 0 && mN->getMemOperand()->getPseudoValue())
5053-
return true;
5054-
Src = mN->getMemOperand()->getValue();
50555048
}
50565049
if (!Src)
50575050
return false;

0 commit comments

Comments
 (0)