[PATCH] D151719: [RISCV] Add special case for (select cc, 1.0, 0.0) to lowerSELECT

[Craig Topper via Phabricator via llvm-commits]

craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:5502
+  if (FPTV && FPFV && FPTV->isExactlyValue(1.0) && FPFV->isExactlyValue(0.0)) {
+    SDValue ZEXT = DAG.getNode(ISD::ZERO_EXTEND, DL, XLenVT, CondV);
+    return DAG.getNode(ISD::SINT_TO_FP, DL, VT, ZEXT);
----------------
This ZERO_EXTEND shoudn't be needed. The condition must be a legal integer type so it could only be XLenVT.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151719