From 06e323a6eeff2652342d96078139f24e39b656e0 Mon Sep 17 00:00:00 2001 From: Benjamin Kramer Date: Fri, 29 Aug 2014 14:38:46 +0000 Subject: [clang-tidy] Add a checker that suggests replacing short/long/long long with fixed-size types. Differential Revision: http://reviews.llvm.org/D5119 llvm-svn: 216726 --- .../test/clang-tidy/google-runtime-int.cpp | 54 ++++++++++++++++++++++ 1 file changed, 54 insertions(+) create mode 100644 clang-tools-extra/test/clang-tidy/google-runtime-int.cpp (limited to 'clang-tools-extra/test/clang-tidy/google-runtime-int.cpp') diff --git a/clang-tools-extra/test/clang-tidy/google-runtime-int.cpp b/clang-tools-extra/test/clang-tidy/google-runtime-int.cpp new file mode 100644 index 00000000000..9fcde079d1e --- /dev/null +++ b/clang-tools-extra/test/clang-tidy/google-runtime-int.cpp @@ -0,0 +1,54 @@ +// RUN: clang-tidy -checks=-*,google-runtime-int %s -- -x c++ 2>&1 | FileCheck %s -implicit-check-not='{{warning:|error:}}' + +long a(); +// CHECK: [[@LINE-1]]:1: warning: consider replacing 'long' with 'int64' + +typedef unsigned long long uint64; // NOLINT + +long b(long = 1); +// CHECK: [[@LINE-1]]:1: warning: consider replacing 'long' with 'int64' +// CHECK: [[@LINE-2]]:8: warning: consider replacing 'long' with 'int64' + +template +void tmpl() { + T i; +} + +short bar(const short, unsigned short) { +// CHECK: [[@LINE-1]]:1: warning: consider replacing 'short' with 'int16' +// CHECK: [[@LINE-2]]:17: warning: consider replacing 'short' with 'int16' +// CHECK: [[@LINE-3]]:24: warning: consider replacing 'unsigned short' with 'uint16' + long double foo = 42; + uint64 qux = 42; + unsigned short port; + + const unsigned short bar = 0; +// CHECK: [[@LINE-1]]:9: warning: consider replacing 'unsigned short' with 'uint16' + long long *baar; +// CHECK: [[@LINE-1]]:3: warning: consider replacing 'long long' with 'int64' + const unsigned short &bara = bar; +// CHECK: [[@LINE-1]]:9: warning: consider replacing 'unsigned short' with 'uint16' + long const long moo = 1; +// CHECK: [[@LINE-1]]:3: warning: consider replacing 'long long' with 'int64' + long volatile long wat = 42; +// CHECK: [[@LINE-1]]:3: warning: consider replacing 'long long' with 'int64' + unsigned long y; +// CHECK: [[@LINE-1]]:3: warning: consider replacing 'unsigned long' with 'uint64' + unsigned long long **const *tmp; +// CHECK: [[@LINE-1]]:3: warning: consider replacing 'unsigned long long' with 'uint64' + unsigned long long **const *&z = tmp; +// CHECK: [[@LINE-1]]:3: warning: consider replacing 'unsigned long long' with 'uint64' + unsigned short porthole; +// CHECK: [[@LINE-1]]:3: warning: consider replacing 'unsigned short' with 'uint16' + +#define l long + l x; + + tmpl(); +// CHECK: [[@LINE-1]]:8: warning: consider replacing 'short' with 'int16' +} + +void qux() { + short port; +// CHECK: [[@LINE-1]]:3: warning: consider replacing 'short' with 'int16' +} -- cgit v1.2.3