[PATCH] D89786: [PowerPC] Avoid unnecessary fadd for unsigned to ppcf128

[Qiu Chaofan via Phabricator via llvm-commits]

qiucf added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/LegalizeFloatTypes.cpp:1638
                                    APInt(NVT.getSizeInBits(), 0)), dl, NVT);
+    HasUnsignedConv = TLI.isOperationLegalOrCustom(ISD::UINT_TO_FP, SrcVT);
+    unsigned ConvOp = (HasUnsignedConv && !isSigned)
----------------
efriedma wrote:
> Does the result of isOperationLegalOrCustom() actually matter here?  Even if it's not legal, the UINT_TO_FP is going to be faster than the alternative.
Ah, that's right.


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

https://reviews.llvm.org/D89786