[PATCH] D50707: NFC: Enforce good formatting across multiple clang-tools-extra files

2018-08-14 Thread Kirill Bobyrev via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rCTE339687: NFC: Enforce good formatting across multiple clang-tools-extra files (authored by omtcyfz, committed by ). Changed prior to commit: https://reviews.llvm.org/D50707?vs=160605&id=160607#toc Rep

[PATCH] D50707: NFC: Enforce good formatting across multiple clang-tools-extra files

2018-08-14 Thread Kirill Bobyrev via Phabricator via cfe-commits
kbobyrev updated this revision to Diff 160605. kbobyrev added a comment. 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. https://reviews.llvm.org/D50707 Files:

[PATCH] D50707: NFC: Enforce good formatting across multiple clang-tools-extra files

2018-08-14 Thread Eric Liu via Phabricator via cfe-commits
ioeric added a comment. 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. https://reviews.llvm.org/D50707 ___ cfe-commits mailing list cfe-commit

[PATCH] D50707: NFC: Enforce good formatting across multiple clang-tools-extra files

2018-08-14 Thread Kirill Bobyrev via Phabricator via cfe-commits
kbobyrev created this revision. kbobyrev added reviewers: ioeric, ilya-biryukov. kbobyrev added a project: clang-tools-extra. Herald added subscribers: arphaman, jkorous. This patch improves readability of multiple files in clang-tools-extra and enforces LLVM Coding Guidelines. https://reviews.