[PATCH] [clang-tidy] Implement the include order checker for LLVM.

Benjamin Kramer benny.kra at gmail.com
Thu Jul 31 09:48:27 PDT 2014


================
Comment at: test/clang-tidy/llvm-include-order.cpp:1
@@ +1,2 @@
+// RUN: sed 's#// *[A-Z-]\+:.*#//#' %s > %t.cpp
+// RUN: clang-tidy %t.cpp -fix --checks="-*,llvm-include-order" -- -I %S/Inputs > %t.msg 2>&1
----------------
Alexander Kornienko wrote:
> Why can't we use check_clang_tidy_fix.sh here?
There is currently no way to add additional flags (include path in this case) with the shell script. The other reason is that this test case produces an error (the includes don't exist) which the script would catch.

http://reviews.llvm.org/D4741






More information about the cfe-commits mailing list