[PATCH] D26015: Correctly classify main file includes if there is a prefix added

2017-01-12 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh added a comment. LG, committed as r291767. Repository: rL LLVM https://reviews.llvm.org/D26015 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D26015: Correctly classify main file includes if there is a prefix added

2017-01-12 Thread Alexander Kornienko via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL291767: Correctly classify main file includes if there is a prefix added (authored by alexfh). Changed prior to commit: https://reviews.llvm.org/D26015?vs=84001&id=84121#toc Repository: rL LLVM http

[PATCH] D26015: Correctly classify main file includes if there is a prefix added

2017-01-11 Thread Julian Bangert via Phabricator via cfe-commits
jbangert updated this revision to Diff 84001. jbangert updated the summary for this revision. jbangert added a comment. - Add header to unit test fixture. https://reviews.llvm.org/D26015 Files: clang-tidy/utils/IncludeSorter.cpp unittests/clang-tidy/IncludeInserterTest.cpp Index: unittests

[PATCH] D26015: Correctly classify main file includes if there is a prefix added

2017-01-11 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh added a comment. Tests don't pass with this patch applied: $ ninja check-clang-tools [22/23] Running the Clang extra tools' regression tests FAIL: Extra Tools Unit Tests :: clang-tidy/ClangTidyTests/Inclu

[PATCH] D26015: Correctly classify main file includes if there is a prefix added

2017-01-11 Thread Alexander Kornienko via Phabricator via cfe-commits
alexfh accepted this revision. alexfh added a comment. This revision is now accepted and ready to land. LG. Thank you for the fix! https://reviews.llvm.org/D26015 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/