[PATCH] D53482: Add clang-format stability check with FormatTests

2018-10-23 Thread Vladimir Glavnyy via Phabricator via cfe-commits
vglavnyy added a comment. @krasimir thank you for review. The patch code has been updated. I hope this patch will help to start to fix this issue. Probably will be helpful to add an optional debug flag will enable a twice-run checking for every run of clang-format.

[PATCH] D53482: Add clang-format stability check with FormatTests

2018-10-23 Thread Vladimir Glavnyy via Phabricator via cfe-commits
vglavnyy updated this revision to Diff 170663. vglavnyy added a comment. A twice-format problem: the format of format isn't format. This commit surface an instability problem in clang-format at unit-tests level. The patch adds double-checking format method for all test and adds a stub for tests

[PATCH] D53482: Add clang-format stability check with FormatTests

2018-10-21 Thread Vladimir Glavnyy via Phabricator via cfe-commits
vglavnyy created this revision. vglavnyy added reviewers: djasper, krasimir. Herald added a subscriber: cfe-commits. Twice running clang-format may give unstable result for some code samples, for example: https://bugs.llvm.org/show_bug.cgi?id=23728 This commit adds stability check to