[clang-format] Don't rewrite the input file if already formatted

If IntegerLiteralSeparator is set but the integer literals are already formatted, don't rewrite the input file. Fixes #60651. Differential Revision: https://reviews.llvm.org/D143831
llvm · Feb 14, 2023 · b4243bb · b4243bb
1 parent ed3e3ee
commit b4243bb
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/clang/lib/Format/IntegerLiteralSeparatorFixer.cpp b/clang/lib/Format/IntegerLiteralSeparatorFixer.cpp
@@ -132,8 +132,11 @@ IntegerLiteralSeparatorFixer::process(const Environment &Env,
       continue;
     if (Start > 0)
       Location = Location.getLocWithOffset(Start);
-    cantFail(Result.add(tooling::Replacement(SourceMgr, Location, Length,
-                                             format(Text, DigitsPerGroup))));
+    if (const auto &Formatted = format(Text, DigitsPerGroup);
+        Formatted != Text) {
+      cantFail(Result.add(
+          tooling::Replacement(SourceMgr, Location, Length, Formatted)));
+    }
   }
 
   return {Result, 0};