[libcxx-commits] [PATCH] D129195: [libc++] Implement P1423R3 (char8_t backward compatibility remediation)

Thu Jul 14 08:26:52 PDT 2022

h-vetinari added a comment.

In D129195#3652046 <https://reviews.llvm.org/D129195#3652046>, @philnik wrote:

> In D129195#3650796 <https://reviews.llvm.org/D129195#3650796>, @h-vetinari wrote:
>
>> Though you probably know, there's yet another DR on top of P1423 <https://reviews.llvm.org/P1423> that has been accepted (only missing plenary vote): https://github.com/cplusplus/papers/issues/1171
>>
>> Might be worth investigating to not needlessly introduce breakage that will then have to be undone again.
>
> The paper you linked only changes core wording, so it shouldn't make any difference for the library part. Or am I missing something?

It's more likely me who missed something, but I just thought I'd point out that paper since it builds on top of P1423 <https://reviews.llvm.org/P1423> and also bumps that feature macro AFAICT.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129195