[PATCH] D91331: Expand the fp_to_int/int_to_fp/fp_round/fp_extend as libcall for fp128

[Qing Shan Zhang via Phabricator via llvm-commits]

steven.zhang added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp:4380
+    std::pair<SDValue, SDValue> Tmp = TLI.makeLibCall(
+        DAG, LC, TLI.getTypeToTransformTo(*DAG.getContext(), RVT), Op,
+        CallOptions, dl, Chain);
----------------
craig.topper wrote:
> We shouldn't to get call getTypeToTransformTo here. The type is already legal right?
Good catch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91331