[libcxx-commits] [PATCH] D138307: [libc++] Remove default definition of std::char_traits

[Tom Honermann via Phabricator via libcxx-commits]

tahonermann added a comment.

I'm concerned that this change will break "working" code. While the standard does not specify specializations for `signed char` and `unsigned char`, it isn't hard to find cases of people using `basic_string<unsigned char>`. I didn't look hard, but it looks like Emscripten would be broken by this change based on https://github.com/emscripten-core/emscripten/issues/7362.

All major compilers appear to "support" specializations for `signed char` and `unsigned char`: https://godbolt.org/z/h3xG7T48T. There is an old paper that argues for standardizing such specializations: https://wg21.link/n1985.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138307