[libcxx-commits] [PATCH] D104980: [libcxx][NFC] replaces `<utility>` includes with specific headers

Eric Fiselier via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Wed Jul 7 11:37:53 PDT 2021


EricWF requested changes to this revision.
EricWF added a comment.

So we're splitting up the headers (which I don't love, but will hold my tongue for the most part).

The intention is to provide better name hygene for our users. Meaning headers will export _fewer_ names than they do at present.
This is an intentional BREAKING change. Users will have to update their code to include the correct headers before it will compile.

Before we do this we need to:

1. Announce and discuss this change, it's rational, and its benefits to *all libc++ vendors*.
2. Document the list of names removed from every header. This list should be complete.
3. Ensure Clang's error messages tell users _which_ headers to include. Or at least don't tell users to include the wrong (implementation) headers.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104980/new/

https://reviews.llvm.org/D104980



More information about the libcxx-commits mailing list