summaryrefslogtreecommitdiffstats
path: root/clang-tools-extra/test/clang-tidy/llvm-include-order.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Analyze include order on a per-file basis.Zachary Turner2016-08-121-0/+5
| | | | | | | | | | | | | | | | | | | The include order check would get notified of all include directives in a depth-first manner. This created the possibility of an include directive from a header file interfering with the sort order of a set of two distinct blocks from the top level cpp file, if that include directive was on just the right line. With this patch we bucket the include directives by the file in which they appear in and process one bucket at a time, so that directives from different files do not get mixed together into the same list. Reviewed By: alexfh Differential Revision: https://reviews.llvm.org/D23434 llvm-svn: 278546
* Switch check_clang_tidy to argparse and add a -resource-dir argument.Manuel Klimek2015-10-221-1/+1
| | | | | | | -resource-dir can be used to inject non-standard resource dirs via the lit site config. llvm-svn: 251021
* Add %check_clang_tidy and %clang_tidy_diff.Manuel Klimek2015-10-221-1/+1
| | | | | | | | | | With this, site specific lit configs can inject parameters into the test scripts if they need site specific parameters. Next up: enable check_clang_tidy to take a resource dir to enable non-standard locations for builtin includes. llvm-svn: 251010
* Remove garbage. The issue was fixed in r246856.NAKAMURA Takumi2015-09-151-2/+0
| | | | llvm-svn: 247689
* [clang-tidy] Fix llvm-include-order check on Windows.Alexander Kornienko2015-09-041-1/+0
| | | | | | | | | | | IncludeDirectives struct used a StringRef that pointed to a stack variable (SmallString<128> FilenameBuffer from PPDirectives.cpp:1513). http://reviews.llvm.org/D12632 Patch by Marek Kurdej! llvm-svn: 246856
* [clang-tidy] Use a python script instead of a shell script to run clang-tidy ↵Alexander Kornienko2015-08-201-2/+1
| | | | | | | | | | | | | | | | | | | tests. Summary: Add check_clang_tidy.py script that is functionally identical to the check_clang_tidy.py, but should also be functional on windows. I've verified that the script works on linux. Would be nice if folks using Windows could test the patch before I break windows bots ;) Reviewers: chapuni, aaron.ballman Subscribers: cfe-commits Differential Revision: http://reviews.llvm.org/D12180 llvm-svn: 245583
* [clang-tidy] check_clang_tidy_fix.sh -> check_clang_tidy.shAlexander Kornienko2014-10-261-1/+1
| | | | | | | | | | | | | | Summary: Make the script suitable for checking just messages. Move most of the tests to use it. Clean up the tests: shorten messages, insert line numbers, remove unnecessary RUN: lines, etc. Reviewers: klimek Reviewed By: klimek Subscribers: curdeius, cfe-commits Differential Revision: http://reviews.llvm.org/D5989 llvm-svn: 220634
* [clang-tidy] Test username substitution in TODO() check.Alexander Kornienko2014-09-261-1/+1
| | | | | | | | | | | | | | | | | | | Summary: This patch depends on D5501. Check user name substitution more specifically by specifying it via the new -config= option. This also lets clang-tidy tests specify clang-tidy options before the '--' when using check_clang_tidy_fix.sh. Reviewers: klimek Reviewed By: klimek Subscribers: curdeius, cfe-commits Differential Revision: http://reviews.llvm.org/D5502 llvm-svn: 218515
* Suppress clang-tools-extra/test/clang-tidy/llvm-include-order.cpp for ↵NAKAMURA Takumi2014-08-081-0/+3
| | | | | | | | | | | | | | | | | targeting msvc. Investigating. FYI, I can see the order; #include "j.h" #include "i.h" #include "clang-c/c.h" // hi #include "clang/b.h" #include "llvm-c/d.h" // -c #include "llvm/a.h" #include "gtest/foo.h" #include <s.h> llvm-svn: 215179
* [clang-tidy] Implement the include order checker for LLVM.Benjamin Kramer2014-08-071-0/+38
There are still a couple of rough edges in here but it is working fine on LLVM and generates the same results as sort_includes.py if there are no blank lines involved. Differential Revision: http://reviews.llvm.org/D4741 llvm-svn: 215152
OpenPOWER on IntegriCloud