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

[Alexey Samsonov via libc-commits]

================
@@ -2569,6 +2569,10 @@ libc_math_function(
 
 libc_math_function(
     name = "expf",
+    hdrs = [
----------------
vonosmas wrote:

header-only `cc_library` targets in Bazel are never actually "built", and comparing Bazel analysis-time overhead of having more Bazel targets and header-parsing overhead of parsing the same __support/math/expf headers for "expf", "exp" and "expd" is comparing apples to oranges... I suspect that at some point we'll need to restructure the Bazel to make it more scalable anyway (the amount of targets is huge for our downstream integration), but for now I've suggested this change to just make it more consistent - we normally have a dedicated targets for `__support` libraries, plus -- as I say in the comment, we'll need to depend on them from the target exposing `libc/shared/` headers.

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