[llvm-branch-commits] [clang] [llvm] Continuation of fexec-charset (PR #169803)

[Sergei Barannikov via llvm-branch-commits]

================
@@ -207,86 +199,104 @@ clang::analyze_format_string::ParseVectorModifier(FormatStringHandler &H,
   return false;
 }
 
-bool
-clang::analyze_format_string::ParseLengthModifier(FormatSpecifier &FS,
-                                                  const char *&I,
-                                                  const char *E,
-                                                  const LangOptions &LO,
-                                                  bool IsScanf) {
+bool clang::analyze_format_string::ParseLengthModifier(
+    FormatSpecifier &FS, const char *&I, const char *E, const LangOptions &LO,
+    const llvm::TextEncodingConverter &FormatStrConverter, bool IsScanf) {
   LengthModifier::Kind lmKind = LengthModifier::None;
   const char *lmPosition = I;
-  switch (*I) {
-    default:
-      return false;
-    case 'h':
+  switch (FormatStrConverter.convert(*I)) {
----------------
s-barannikov wrote:

Can you create a separate PR that clang-formats all modified functions? It would make the review much easier.


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