[PATCH] D61390: [CodeGen] Add lround/llround builtins

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 6 22:43:58 PDT 2019


craig.topper added inline comments.


================
Comment at: llvm/lib/CodeGen/TargetLoweringBase.cpp:700
   // These library functions default to expand.
-  for (MVT VT : {MVT::f32, MVT::f64, MVT::f128}) {
+  for (MVT VT : {MVT::f32, MVT::f64, MVT::f80, MVT::f128}) {
     setOperationAction(ISD::FCBRT,      VT, Expand);
----------------
Instead of adding f80 here which only X86 needs, just add the new nodes in the same spot as FLOG, FLOG2, FLOG10, etc. in X86ISelLowering.cpp


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

https://reviews.llvm.org/D61390





More information about the llvm-commits mailing list