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
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
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
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
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/
