[libc-commits] [libc] [llvm] [libc][math] Refactor sinf16 implementation to header-only in src/__support/math folder. (PR #178062)

[Muhammad Bassiouni via libc-commits]

================
@@ -3507,6 +3507,17 @@ libc_support_library(
     ],
 )
 
+libc_support_library(
+    name = "__support_math_sinf16",
+    hdrs = ["src/__support/math/sinf16.h"],
+    deps = [
+        ":__support_fputil_nearest_integer",
+        ":__support_fputil_polyeval",
+        ":__support_math_sincosf16_utils",
+        ":llvm_libc_macros_float16_macros",
----------------
bassiounix wrote:

I think for each and every include we want to add its deps

```cc
#include "hdr/errno_macros.h"
#include "hdr/fenv_macros.h"
#include "sincosf16_utils.h"
#include "src/__support/FPUtil/FEnvImpl.h"
#include "src/__support/FPUtil/FPBits.h"
#include "src/__support/FPUtil/cast.h"
#include "src/__support/FPUtil/except_value_utils.h"
#include "src/__support/FPUtil/multiply_add.h"
#include "src/__support/macros/optimization.h"
```

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