[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