[libc-commits] [libc] [llvm] [libc][math] Refactor fdim family to header-only (PR #182190)

[Muhammad Bassiouni via libc-commits]

================
@@ -751,6 +751,57 @@ add_header_library(
     libc.src.__support.FPUtil.generic.add_sub
     libc.src.__support.macros.config
 )
+add_header_library(
+  fdim
+  HDRS
+    fdim.h
+  DEPENDS
+    libc.src.__support.FPUtil.basic_operations
+    libc.src.__support.macros.config
+)
+add_header_library(
+  fdimbf16
+  HDRS
+    fdimbf16.h
+  DEPENDS
+    libc.src.__support.FPUtil.basic_operations
+    libc.src.__support.FPUtil.bfloat16
+    libc.src.__support.macros.config
+)
+add_header_library(
+  fdimf
+  HDRS
+    fdimf.h
+  DEPENDS
+    libc.src.__support.FPUtil.basic_operations
+    libc.src.__support.macros.config
+)
+add_header_library(
+  fdimf128
+  HDRS
+    fdimf128.h
+  DEPENDS
+    libc.include.llvm-libc-types.float128
+    libc.src.__support.FPUtil.basic_operations
+    libc.src.__support.macros.config
+)
+add_header_library(
+  fdimf16
+  HDRS
+    fdimf16.h
+  DEPENDS
+    libc.include.llvm-libc-macros.float16_macros
+    libc.src.__support.FPUtil.basic_operations
+    libc.src.__support.macros.config
+)
+add_header_library(
+  fdiml
+  HDRS
+    fdiml.h
+  DEPENDS
+    libc.src.__support.FPUtil.basic_operations
+    libc.src.__support.macros.config
+)
----------------
bassiounix wrote:

```suggestion
add_header_library(
  fdim
  HDRS
    fdim.h
  DEPENDS
    libc.src.__support.FPUtil.basic_operations
    libc.src.__support.macros.config
)

add_header_library(
  fdimbf16
  HDRS
    fdimbf16.h
  DEPENDS
    libc.src.__support.FPUtil.basic_operations
    libc.src.__support.FPUtil.bfloat16
    libc.src.__support.macros.config
)

add_header_library(
  fdimf
  HDRS
    fdimf.h
  DEPENDS
    libc.src.__support.FPUtil.basic_operations
    libc.src.__support.macros.config
)

add_header_library(
  fdimf128
  HDRS
    fdimf128.h
  DEPENDS
    libc.include.llvm-libc-types.float128
    libc.src.__support.FPUtil.basic_operations
    libc.src.__support.macros.config
)

add_header_library(
  fdimf16
  HDRS
    fdimf16.h
  DEPENDS
    libc.include.llvm-libc-macros.float16_macros
    libc.src.__support.FPUtil.basic_operations
    libc.src.__support.macros.config
)

add_header_library(
  fdiml
  HDRS
    fdiml.h
  DEPENDS
    libc.src.__support.FPUtil.basic_operations
    libc.src.__support.macros.config
)
```

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