[PATCH] D93031: Enable fexec-charset option

Abhina Sree via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 29 11:34:15 PST 2020


abhina.sreeskantharajan updated this revision to Diff 313990.
abhina.sreeskantharajan added a comment.

Thanks for the review! I've addressed most of the comments but I still need to work on the translation issues in CharLiteralParser that was kindly pointed out by Tom and Richard. Here are the summary of changes in this patch:

- Removed TranslationState as a member of StringLiteralParser and pass it as an argument instead
- Added an assertion for wide character translation instead of translating them to the system charset
- Invalid char escapes are changed to '?' and then translated
- Updated testcases as requested


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93031

Files:
  clang/include/clang/Basic/LangOptions.h
  clang/include/clang/Basic/TokenKinds.h
  clang/include/clang/Driver/Options.td
  clang/include/clang/Lex/LiteralSupport.h
  clang/include/clang/Lex/LiteralTranslator.h
  clang/include/clang/Lex/Preprocessor.h
  clang/lib/Driver/ToolChains/Clang.cpp
  clang/lib/Frontend/CompilerInstance.cpp
  clang/lib/Frontend/CompilerInvocation.cpp
  clang/lib/Lex/CMakeLists.txt
  clang/lib/Lex/LiteralSupport.cpp
  clang/lib/Lex/LiteralTranslator.cpp
  clang/lib/Lex/Preprocessor.cpp
  clang/test/CodeGen/systemz-charset.c
  clang/test/CodeGen/systemz-charset.cpp
  clang/test/Driver/cl-options.c
  clang/test/Driver/clang_f_opts.c
  llvm/include/llvm/ADT/Triple.h
  llvm/lib/Support/Triple.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D93031.313990.patch
Type: text/x-patch
Size: 34257 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201229/b5d41d22/attachment-0001.bin>


More information about the llvm-commits mailing list