[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