[libc-commits] [libc] [libc][math][c23] Add ffma{d, l, f128} and fdiv{d, l, f128} C23 math functions #101089 (PR #101253)
via libc-commits
libc-commits at lists.llvm.org
Wed Jul 31 09:25:49 PDT 2024
https://github.com/overmighty commented:
Since you're adding smoke tests that use DivTest.h, could you update the includes in libc/test/src/math/smoke/DivTest.h like this?
```diff
diff --git a/libc/test/src/math/smoke/DivTest.h b/libc/test/src/math/smoke/DivTest.h
index b30fc17aac1d..666179628c55 100644
--- a/libc/test/src/math/smoke/DivTest.h
+++ b/libc/test/src/math/smoke/DivTest.h
@@ -9,8 +9,8 @@
#ifndef LLVM_LIBC_TEST_SRC_MATH_SMOKE_DIVTEST_H
#define LLVM_LIBC_TEST_SRC_MATH_SMOKE_DIVTEST_H
+#include "hdr/errno_macros.h"
#include "hdr/fenv_macros.h"
-#include "src/__support/FPUtil/BasicOperations.h"
#include "test/UnitTest/FEnvSafeTest.h"
#include "test/UnitTest/FPMatcher.h"
#include "test/UnitTest/RoundingModeUtils.h"
```
And the dependencies of smoke tests for `f16div*` functions in CMakeLists.txt accordingly. For example:
```diff
diff --git a/libc/test/src/math/smoke/CMakeLists.txt b/libc/test/src/math/smoke/CMakeLists.txt
index f940b5f918f6..0667047d125d 100644
--- a/libc/test/src/math/smoke/CMakeLists.txt
+++ b/libc/test/src/math/smoke/CMakeLists.txt
@@ -3874,8 +3874,8 @@ add_fp_unittest(
HDRS
DivTest.h
DEPENDS
+ libc.hdr.errno_macros
libc.hdr.fenv_macros
- libc.src.__support.FPUtil.basic_operations
libc.src.math.f16div
)
```
https://github.com/llvm/llvm-project/pull/101253
More information about the libc-commits
mailing list