[PATCH] D153621: [Clang] Correctly handle $, @, and ` when represented as UCN

Shafik Yaghmour via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jun 26 09:27:33 PDT 2023


shafik added inline comments.


================
Comment at: clang/test/Lexer/utf8-char-literal.cpp:22
 char b = u8'\x80';   // ok
-char c = u8'\u0080'; // expected-error {{universal character name refers to a control character}}
+char c = u8'\u0000'; // ok
 char d = u8'\u1234'; // expected-error {{character too large for enclosing character literal type}}
----------------
Why change this test line?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153621



More information about the cfe-commits mailing list