[PATCH] D155623: [AArch64][NFC] Expand SLEEF coverage for ReplaceWithVeclib testing
mgabka via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Jul 21 04:59:34 PDT 2023
mgabka added a comment.
OK then I think we are still missing:
llrint
llround
lrint
nearbyint
as those are defined in libm.
Also in my opinion the commit message should directly reference libm not use "math intrinsics" as abs is also a math operation, but it is not defined in libm
================
Comment at: llvm/test/CodeGen/AArch64/replace-intrinsics-with-veclib-sleef.ll:186
+
+define <2 x double> @llvm_log10_f64(<2 x double> %in) {
+; CHECK-LABEL: @llvm_log10_f64(
----------------
nit: I think that log10 should be before log2 if we are aiming to keep alphabetical order
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D155623/new/
https://reviews.llvm.org/D155623
More information about the llvm-commits
mailing list