[PATCH] D87169: SVML support for log10, sqrt

[Hongtao Yu via Phabricator via llvm-commits]

hoy added inline comments.


================
Comment at: llvm/include/llvm/Analysis/VecFuncs.def:312
+
+TLI_DEFINE_VECFUNC("llvm.sqrt.f64", "__svml_sqrt2", 2)
+TLI_DEFINE_VECFUNC("llvm.sqrt.f64", "__svml_sqrt4", 4)
----------------
craig.topper wrote:
> I don't think we want to use SVML for llvm.sqrt. We have a hardware instruction for that.
Good point. We have the SQRTPS/SQRTPD AVX instructions for that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D87169