This is an archive of the discontinued LLVM Phabricator instance.

[clang-format] Never analyze insert/remove braces in the same pass
ClosedPublic

Authored by owenpan on Jun 13 2022, 12:50 PM.

Details

Summary

Turn off RemoveBracesLLVM while analyzing InsertBraces and vice versa to avoid potential interference of each other and better the performance.

Diff Detail

Event Timeline

owenpan created this revision.Jun 13 2022, 12:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 13 2022, 12:50 PM
owenpan requested review of this revision.Jun 13 2022, 12:50 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 13 2022, 12:50 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
This revision is now accepted and ready to land.Jun 14 2022, 1:07 PM
curdeius accepted this revision.Jun 14 2022, 10:27 PM
This revision was landed with ongoing or failed builds.Jun 14 2022, 10:43 PM
This revision was automatically updated to reflect the committed changes.