[clang] ff11354 - [clang-format][NFC] Use the newly added AllowLiteralDigitSeparator (#185165)

via cfe-commits cfe-commits at lists.llvm.org
Sat Mar 7 12:47:17 PST 2026


Author: owenca
Date: 2026-03-07T12:47:12-08:00
New Revision: ff11354911024ff443145e2a502e83f42066418a

URL: https://github.com/llvm/llvm-project/commit/ff11354911024ff443145e2a502e83f42066418a
DIFF: https://github.com/llvm/llvm-project/commit/ff11354911024ff443145e2a502e83f42066418a.diff

LOG: [clang-format][NFC] Use the newly added AllowLiteralDigitSeparator (#185165)

Use LangOptions::AllowLiteralDigitSeparator added in #184235 for the
IntegerLiteralSeparator option.

Added: 
    

Modified: 
    clang/lib/Format/IntegerLiteralSeparatorFixer.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/Format/IntegerLiteralSeparatorFixer.cpp b/clang/lib/Format/IntegerLiteralSeparatorFixer.cpp
index 5f40100c2b968..eea9fcc56af63 100644
--- a/clang/lib/Format/IntegerLiteralSeparatorFixer.cpp
+++ b/clang/lib/Format/IntegerLiteralSeparatorFixer.cpp
@@ -44,6 +44,8 @@ static Base getBase(StringRef IntegerLiteral) {
 std::pair<tooling::Replacements, unsigned>
 IntegerLiteralSeparatorFixer::process(const Environment &Env,
                                       const FormatStyle &Style) {
+  const auto LangOpts = getFormattingLangOpts(Style);
+
   switch (Style.Language) {
   case FormatStyle::LK_CSharp:
   case FormatStyle::LK_Java:
@@ -53,11 +55,10 @@ IntegerLiteralSeparatorFixer::process(const Environment &Env,
   case FormatStyle::LK_C:
   case FormatStyle::LK_Cpp:
   case FormatStyle::LK_ObjC:
-    if (Style.Standard >= FormatStyle::LS_Cpp14) {
-      Separator = '\'';
-      break;
-    }
-    [[fallthrough]];
+    if (!LangOpts.AllowLiteralDigitSeparator)
+      return {};
+    Separator = '\'';
+    break;
   default:
     return {};
   }
@@ -94,7 +95,6 @@ IntegerLiteralSeparatorFixer::process(const Environment &Env,
   AffectedRangeManager AffectedRangeMgr(SourceMgr, Env.getCharRanges());
 
   const auto ID = Env.getFileID();
-  const auto LangOpts = getFormattingLangOpts(Style);
   Lexer Lex(ID, SourceMgr.getBufferOrFake(ID), SourceMgr, LangOpts);
   Lex.SetCommentRetentionState(true);
 


        


More information about the cfe-commits mailing list