REPOSITORY
rL LLVM
================
Comment at: clang-tools-extra/trunk/clang-tidy/google/GoogleTidyModule.cpp:45
@@ -43,1 +44,3 @@
CheckFactories.addCheckFactory(
+ "google-runtime-int",
+ new ClangTidyCheckFactory<runtime::IntegerTypesCheck>());
----------------
The name should be more explicit.
================
Comment at: clang-tools-extra/trunk/clang-tidy/google/IntegerTypesCheck.cpp:20
@@ +19,3 @@
+namespace ast_matchers {
+const internal::VariadicDynCastAllOfMatcher<Decl, TypedefDecl> typedefDecl;
+} // namespace ast_matchers
----------------
Move this to ASTMatchers.h
================
Comment at: clang-tools-extra/trunk/test/clang-tidy/google-runtime-int.cpp:4
@@ +3,3 @@
+long a();
+// CHECK: [[@LINE-1]]:1: warning: consider replacing 'long' with 'int64'
+
----------------
This will fail on some platforms, right? "long" is not 64 bits everywhere
(which is the point of this clang-tidy check).
http://reviews.llvm.org/D5119
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
