This is an archive of the discontinued LLVM Phabricator instance.

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

Authored by owenpan on Feb 11 2023, 4:02 PM.

Details

Summary

If IntegerLiteralSeparator is set but the integer literals are already formatted, don't rewrite the input file.

Fixes https://github.com/llvm/llvm-project/issues/60651.

Diff Detail

Event Timeline

owenpan created this revision.Feb 11 2023, 4:02 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 11 2023, 4:02 PM
owenpan requested review of this revision.Feb 11 2023, 4:02 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 11 2023, 4:02 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
MyDeveloperDay accepted this revision.Feb 14 2023, 2:51 AM
This revision is now accepted and ready to land.Feb 14 2023, 2:51 AM
This revision was landed with ongoing or failed builds.Feb 14 2023, 2:28 PM
This revision was automatically updated to reflect the committed changes.