tbaeder marked an inline comment as done.
tbaeder added inline comments.

================
Comment at: clang/test/Lexer/utf8-char-literal.cpp:24
+char g = u8'\x80';          // expected-warning {{implicit conversion from 
'int' to 'char' changes value from 128 to -128}}
 #endif
----------------
tahonermann wrote:
> We should also exercise the preprocessor with something like this:
>   #if u8'\xff' != 0xff
>   #error uh oh
>   #endif
> 
> Hmm, this currently fails for C++20 for both Clang and gcc unless 
> `-funsigned-char` is passed. That seems wrong. 
> https://godbolt.org/z/Tb7z85ToG. MSVC gets this wrong too, but I think for a 
> different reason; see the implementation impact section of [[ 
> https://wg21.link/p2029 | P2029 ]] if curious.
This also fails in C2x.


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

https://reviews.llvm.org/D119221

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to