[PATCH] D90738: [RISCV] Support Zfh half-precision floating-point extension.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 24 18:44:22 PST 2020


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:2076
     }
     Val = DAG.getNode(ISD::BITCAST, DL, VA.getValVT(), Val);
     break;
----------------
If you're going to make changes to this patch. Can you move this into a final else and pull out the "break;" Similar to the code in convertValVTToLocVT. Sorry I missed it earlier.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90738



More information about the llvm-commits mailing list