[PATCH] D93031: Enable fexec-charset option

Jan Svoboda via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 1 00:00:48 PST 2021


jansvoboda11 added inline comments.


================
Comment at: clang/include/clang/Driver/Options.td:3583
 
+def fexec_charset : Separate<["-"], "fexec-charset">, MetaVarName<"<charset>">,
+  HelpText<"Set the execution <charset> for string and character literals. "
----------------
Could you switch to the option marshalling infrastructure? https://clang.llvm.org/docs/InternalsManual.html#adding-new-command-line-option
Adding `MarshallingInfoString<LangOpts<"ExecCharset">>` here should do the trick. You can then delete the option parsing in `CompilerInvocation.cpp`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D93031



More information about the llvm-commits mailing list