[PATCH] D69275: Add constrained int->FP intrinsics

Kevin P. Neal via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 10 10:46:23 PST 2019


kpn marked an inline comment as done.
kpn added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp:2403
     // final result
-    SDValue Result = DAG.getFPExtendOrRound(Sub, dl, DestVT);
+    SDValue Result, NewChain;
+    if (Node->isStrictFPOpcode()) {
----------------
uweigand wrote:
> NewChain seems superfluous?
Right, fixed now. Thanks!


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D69275/new/

https://reviews.llvm.org/D69275





More information about the llvm-commits mailing list