[libcxx-commits] [libcxx] [libc++] add commonly used aliases in <__config> and remove <cstddef> includes (PR #70598)
Louis Dionne via libcxx-commits
libcxx-commits at lists.llvm.org
Mon Oct 30 21:53:12 PDT 2023
https://github.com/ldionne requested changes to this pull request.
I pretty strongly disagree with this on a conceptual basis. `__config` is meant to contain configuration stuff for the library, not actual declarations.
What concrete problem are we solving with this patch? What kind of improvement are we seeing? Basically all the code in the world includes `<cstddef>` transitively one way or another, so I would be surprised if this actually ended up providing significant compile-time benefits.
https://github.com/llvm/llvm-project/pull/70598
More information about the libcxx-commits
mailing list