[PATCH] D130370: [llvm] Always use TargetConstant for FP_ROUND ISD Nodes
Craig Topper via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 26 09:04:40 PDT 2022
craig.topper added inline comments.
================
Comment at: llvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp:15757
return DAG.getNode(ISD::FP_ROUND, DL, VT, N0.getOperand(0),
- DAG.getIntPtrConstant(NIsTrunc && N0IsTrunc, DL));
+ DAG.getTargetConstant(NIsTrunc && N0IsTrunc, DL, TLI.getPointerTy(DAG.getDataLayout())));
}
----------------
lines should wrap before 80 columns. Use clang-format.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D130370/new/
https://reviews.llvm.org/D130370
More information about the llvm-commits
mailing list