[libcxx-commits] [PATCH] D120996: [libc++] Remove extension to support allocator<const T>

[Fangrui Song via Phabricator via libcxx-commits]

MaskRay added a comment.

Thanks for giving a transition period:) We are actively working on the migration.

The good news is that `std::allocator<const` is extremely rare. The breakage mostly comes from container usage like: [[ https://codesearch.debian.net/search?q=std%3A%3Avector%3Cconst%5B+%5D%2B%5Cw%2B%28%3C%5B%5E%3E*%5D%2B%3E%5B+%5D*%29%3F%3E&literal=0 | `std::vector<const[ ]+\w+(<[^>*]+>[ ]*)?>` ]], which is rare as well because https://gcc.gnu.org/bugzilla/show_bug.cgi?id=48101 libstdc++ since 8.0 has static_assert with diagnostics like `error: static assertion failed: std::vector must have a non-const, non-volatile value_type`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D120996