This is an archive of the discontinued LLVM Phabricator instance.

NFC: Enforce good formatting across multiple clang-tools-extra files
ClosedPublic

Authored by kbobyrev on Aug 14 2018, 8:01 AM.

Diff Detail

Event Timeline

kbobyrev created this revision.Aug 14 2018, 8:01 AM

Fixing header guards and file comments is fine, but I'm not a fan of reformatting-only changes in source code as they tends to make git blame harder.

kbobyrev updated this revision to Diff 160605.Aug 14 2018, 8:53 AM

I have updated the patch so that it only affects comments, header guards and inserts few newlines. Actual source code is not affected so that git blame log could be less cryptic.

ioeric accepted this revision.Aug 14 2018, 8:58 AM
This revision is now accepted and ready to land.Aug 14 2018, 8:58 AM
This revision was automatically updated to reflect the committed changes.