[PATCH] D101759: [PowerPC] Scalar IBM MASS library conversion pass

[Bardia Mahjour via Phabricator via llvm-commits]

bmahjour added inline comments.


================
Comment at: llvm/include/llvm/Analysis/ScalarFuncs.def:17
+
+TLI_DEFINE_SCALAR_MASS_FUNC("acosf", "__xl_acosf_finite")
+TLI_DEFINE_SCALAR_MASS_FUNC("__acosf_finite", "__xl_acosf_finite")
----------------
shouldn't these map from llvm.* intrinsics to mass entry points as well?


================
Comment at: llvm/lib/Target/PowerPC/PPCISelLowering.cpp:1361
   setLibcallName(RTLIB::FMA_F128, "fmaf128");
+  if (TM.Options.PPCGenScalarMASSEntries && TM.Options.UnsafeFPMath) {
+    setLibcallName(RTLIB::COS_F64, "__xl_cos_finite");
----------------
why are these being handled here instead of `PPCGenScalarMASSEntries.cpp`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101759