Skip to content
12 changes: 10 additions & 2 deletions llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -3980,8 +3980,16 @@ void SelectionDAGBuilder::visitSIToFP(const User &I) {
}

void SelectionDAGBuilder::visitPtrToAddr(const User &I) {
// FIXME: this is not correct for pointers with addr width != pointer width
visitPtrToInt(I);
const auto &TLI = DAG.getTargetLoweringInfo();
const DataLayout &DL = DAG.getDataLayout();
// ptrtoaddr is equivalent to a truncate of ptrtoint to address/index width
auto Op0 = I.getOperand(0);
SDValue N = getValue(Op0);
// By definition the type of the ptrtoaddr must be equal to the address type.
assert(I.getType() == DL.getAddressType(Op0->getType()));
EVT AddrVT = TLI.getValueType(DL, I.getType());
N = DAG.getNode(ISD::TRUNCATE, getCurSDLoc(), AddrVT, N);
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If these are the same type, why do you need the truncate?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While most targets will have them be the same, the Op0 type can be wider. This is the case e.g. for the AMD GPU buffer pointers.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

added a comment, hopefully makes sense now.

setValue(&I, N);
}

void SelectionDAGBuilder::visitPtrToInt(const User &I) {
Expand Down
Loading