[PATCH] D48149: Expand constrained FP POWI

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 13 13:48:18 PDT 2018


craig.topper added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/LegalizeVectorOps.cpp:729
     return ExpandCTTZ_ZERO_UNDEF(Op);
   case ISD::STRICT_FSQRT:
   case ISD::STRICT_FMA:
----------------
Should STRICT_FMUL/DIV/ADD/SUB be here even though I don't know how to test them? If some target did it someday, we would end up the default case which would do the wrong thing.


Repository:
  rL LLVM

https://reviews.llvm.org/D48149





More information about the llvm-commits mailing list