tahonermann added inline comments.

================
Comment at: clang/lib/Lex/Lexer.cpp:3323
     if (Diagnose)
       Diag(StartPtr, diag::warn_ucn_escape_incomplete);
     return llvm::None;
----------------
I was able to confirm that `StartPtr` does point to `N`. See the diagnostic 
generated at https://godbolt.org/z/qnajcMeso; the diagnostic caret points to 
`N` instead of `\`.
  <source>:1:2: warning: incomplete delimited universal character name; 
treating as '\' 'N' '{' identifier [-Wunicode]
  \N{abc
   ^


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138861

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

Reply via email to