This is an archive of the discontinued LLVM Phabricator instance.

[clang-format] NFC - clang-format the FormatTests
ClosedPublic

Authored by MyDeveloperDay on Apr 30 2020, 2:02 PM.

Details

Summary

Ensure the clang-format unit tests are themselves clang-formatted

Having areas of the llvm code which are clang-format clean, give us more areas to run new clang-format binaries on ensuring we haven't broken anything.

It seems to me we SHOULD have this clang-formatted at a minimum, otherwise how can we expect others to use clang-format if we "don't eat our own dogfood", also if the tests are dependent on the formatting of the code then that would also be bad!

Diff Detail

Event Timeline

MyDeveloperDay created this revision.Apr 30 2020, 2:02 PM
sammccall accepted this revision.Apr 30 2020, 4:11 PM

Makes sense to me!

This revision is now accepted and ready to land.Apr 30 2020, 4:11 PM
This revision was automatically updated to reflect the committed changes.