[libc-commits] [libc] [llvm] [libc][math] Refactor log to header-only shared math (PR #175395)

[Muhammad Bassiouni via libc-commits]

================
@@ -1025,6 +1025,33 @@ add_header_library(
     libc.src.__support.FPUtil.multiply_add
 )
 
+add_header_library(
+  log 
+  HDRS
+    log.h
+  DEPENDS
+    .log_range_reduction
+    libc.src.__support.FPUtil.double_double
+    libc.src.__support.FPUtil.dyadic_float
+    libc.src.__support.FPUtil.fenv_impl
+    libc.src.__support.FPUtil.fp_bits
+    libc.src.__support.FPUtil.multiply_add
+    libc.src.__support.FPUtil.polyeval
+    libc.src.__support.integer_literals
+    libc.src.__support.macros.optimization
+    libc.src.__support.math.common_constants
+)
+
+add_header_library(
+  log_range_reduction
+  HDRS
+    log_range_reduction.h
+  DEPENDS
+    libc.src.__support.fputil.dyadic_float
----------------
bassiounix wrote:

The fails are caused by missing the correct dependency.
```suggestion
    libc.src.__support.FPUtil.dyadic_float
```

Please be careful next time. I'm cleaning it up.

@CarvedCoder 

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