[clang] [llvm] Enable fexec-charset option (PR #138895)

Abhina Sree via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 5 11:36:36 PDT 2025


================
@@ -246,18 +249,19 @@ class StringLiteralParser {
   StringLiteralEvalMethod EvalMethod;
 
 public:
-  StringLiteralParser(ArrayRef<Token> StringToks, Preprocessor &PP,
-                      StringLiteralEvalMethod StringMethod =
-                          StringLiteralEvalMethod::Evaluated);
+  StringLiteralParser(
+      ArrayRef<Token> StringToks, Preprocessor &PP,
+      StringLiteralEvalMethod StringMethod = StringLiteralEvalMethod::Evaluated,
+      ConversionAction Action = ToExecCharset);
----------------
abhina-sree wrote:

So I put up a table on my old RFC https://discourse.llvm.org/t/rfc-enabling-fexec-charset-support-to-llvm-and-clang-reposting/71512 which shows the different encoding needed depending on the context. So not all strings will be translated to the execution encoding. A separate PR after this one will be needed to ensure we use the correct encoding depending on the context.

https://github.com/llvm/llvm-project/pull/138895


More information about the llvm-commits mailing list