[libcxx-commits] [libcxx] [libc++] Only include math implementation headers from `math.h`. (PR #109073)

Nikolas Klauser via libcxx-commits libcxx-commits at lists.llvm.org
Wed Sep 18 01:35:51 PDT 2024


https://github.com/philnik777 requested changes to this pull request.

IMO `math.h` shouldn't include the special functions, since they don't belong there. Also, what is the failure mode here?
This code works perfectly fine:
```c++
void frexp();
#define frexp
#undef frexp
void func() {
  frexp();
}
```
I'm pretty sure we had a very similar patch previously and the failure mode never got explained.


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


More information about the libcxx-commits mailing list