[libc-commits] [libc] [libc][math][c23] Add exp2f16 C23 math function (PR #101217)

via libc-commits libc-commits at lists.llvm.org
Tue Jul 30 11:49:14 PDT 2024


================
@@ -164,6 +164,17 @@ add_perf_binary(
     -fno-builtin
 )
 
+add_perf_binary(
+  exp2f16_perf
+  SRCS
+    exp2f16_perf.cpp
+  DEPENDS
+    .single_input_single_output_diff
+    libc.src.math.exp2f16
+  COMPILE_OPTIONS
+    -fno-builtin
----------------
overmighty wrote:

All the math performance tests, except one, benchmark LLVM libc functions against respective system libc functions, so all of them except that one have `-fno-builtin`. I don't know why fine-grained `-fno-builtin-<function>` flags weren't used.

In this case, since the system libc shouldn't have `float16` math functions yet, I benchmark against a placeholder function for now. It would probably be better if at some point we just rewrote our performance testing code to use Google Benchmark, and also allow us to benchmark a single function instead of comparing two, while we're at it.

https://github.com/llvm/llvm-project/pull/101217


More information about the libc-commits mailing list