[libc-commits] [libc] [libc][math][c23] Add fadd{l, f128} C23 math functions (PR #102531)

[via libc-commits]

================
@@ -155,6 +155,34 @@ add_fp_unittest(
 
 )
 
+add_fp_unittest(
+  faddl_test
+  NEED_MPFR
+  SUITE
+    libc-math-unittests
+  SRCS
+    faddl_test.cpp
+  HDRS
+    AddTest.h
+  DEPENDS
+    libc.src.math.faddl
+    libc.src.__support.FPUtil.basic_operations
+)
----------------
overmighty wrote:

No, it doesn't use `fputil::generic::add_sub` directly, either. It included BasicOperations.h to use `fputil::getpayload`, but use of that has since been removed.

I use Neovim with Clangd, and I get a warning saying "Included header BasicOperations.h is not used directly". I get the same for errno_macros.h, fenv_macros.h, and Test.h, but these are proxy headers, and we do use them as they are intended to be.

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