Add timed out test files to tidy_timeout_srcs
* Timed out runs do not show any warning messages.
* These test files cannot finish clang-tidy runs with
the following settings:
TIDY_TIMEOUT=90
WITH_TIDY=1
CLANG_ANALYZER_CHECKS=1
* When TIDY_TIMEOUT is set, in Android continuous builds,
tidy_timeout_srcs files will not be compiled by clang-tidy.
When developers build locally without TIDY_TIMEOUT,
tidy_timeout_srcs files will be compiled.
* Some of these test modules may be split into smaller ones,
or disable some time consuming checks, and then
enable clang-tidy to run within limited time.
Bug: 201099167
Test: make droid tidy-art_subset
Change-Id: I2b26507d6d8843fadad1dfe2406db6c412bd5610
diff --git a/cmdline/Android.bp b/cmdline/Android.bp
index bf00a85..91c8ccd 100644
--- a/cmdline/Android.bp
+++ b/cmdline/Android.bp
@@ -38,6 +38,7 @@
art_cc_defaults {
name: "art_cmdline_tests_defaults",
+ tidy_timeout_srcs: ["cmdline_parser_test.cc"],
srcs: ["cmdline_parser_test.cc"],
}
diff --git a/compiler/Android.bp b/compiler/Android.bp
index b53707a..de98fdb 100644
--- a/compiler/Android.bp
+++ b/compiler/Android.bp
@@ -371,6 +371,15 @@
":art-gtest-jars-Interfaces",
":art-gtest-jars-MyClassNatives",
],
+ tidy_timeout_srcs: [
+ "jni/jni_compiler_test.cc",
+ "optimizing/codegen_test.cc",
+ "optimizing/constant_folding_test.cc",
+ "optimizing/execution_subgraph_test.cc",
+ "optimizing/induction_var_range_test.cc",
+ "optimizing/load_store_elimination_test.cc",
+ "optimizing/ssa_test.cc",
+ ],
srcs: [
"debug/dwarf/dwarf_test.cc",
"debug/src_map_elem_test.cc",
@@ -517,6 +526,10 @@
defaults: [
"art_gtest_defaults",
],
+ tidy_timeout_srcs: [
+ "utils/x86/assembler_x86_test.cc",
+ "utils/x86_64/assembler_x86_64_test.cc",
+ ],
codegen: {
arm: {
srcs: [
diff --git a/dex2oat/Android.bp b/dex2oat/Android.bp
index e5c2f12..a89c62f 100644
--- a/dex2oat/Android.bp
+++ b/dex2oat/Android.bp
@@ -483,6 +483,12 @@
":art-gtest-jars-VerifierDepsMulti",
":art-gtest-jars-VerifySoftFailDuringClinit",
],
+ tidy_timeout_srcs: [
+ "dex2oat_test.cc",
+ "verifier_deps_test.cc",
+ "linker/arm/relative_patcher_thumb2_test.cc",
+ "linker/arm64/relative_patcher_arm64_test.cc",
+ ],
srcs: [
"dex2oat_test.cc",
"dex2oat_vdex_test.cc",
diff --git a/libartbase/Android.bp b/libartbase/Android.bp
index 3c00949..f937523 100644
--- a/libartbase/Android.bp
+++ b/libartbase/Android.bp
@@ -314,6 +314,10 @@
art_cc_defaults {
name: "art_libartbase_tests_defaults",
+ tidy_timeout_srcs: [
+ "base/bit_utils_test.cc",
+ "base/intrusive_forward_list_test.cc",
+ ],
srcs: [
"arch/instruction_set_test.cc",
"base/arena_allocator_test.cc",
diff --git a/libdexfile/Android.bp b/libdexfile/Android.bp
index bc722d8..fdc57d0 100644
--- a/libdexfile/Android.bp
+++ b/libdexfile/Android.bp
@@ -27,6 +27,9 @@
name: "libdexfile_defaults",
defaults: ["art_defaults"],
host_supported: true,
+ tidy_timeout_srcs: [
+ "dex/dex_file_verifier.cc",
+ ],
srcs: [
"dex/art_dex_file_loader.cc",
"dex/compact_dex_file.cc",
@@ -262,6 +265,9 @@
art_cc_defaults {
name: "art_libdexfile_tests_defaults",
+ tidy_timeout_srcs: [
+ "dex/dex_file_verifier_test.cc",
+ ],
srcs: [
"dex/art_dex_file_loader_test.cc",
"dex/class_accessor_test.cc",
diff --git a/libnativeloader/Android.bp b/libnativeloader/Android.bp
index 7352be2..314e92e 100644
--- a/libnativeloader/Android.bp
+++ b/libnativeloader/Android.bp
@@ -153,6 +153,9 @@
"art_standalone_test_defaults",
"libnativeloader-test-defaults",
],
+ tidy_timeout_srcs: [
+ "native_loader_test.cpp",
+ ],
srcs: [
"native_loader_api_test.c",
"native_loader_test.cpp",
diff --git a/profman/Android.bp b/profman/Android.bp
index 0317fe6..b231499 100644
--- a/profman/Android.bp
+++ b/profman/Android.bp
@@ -165,6 +165,7 @@
data: [
":art-gtest-jars-ProfileTestMultiDex",
],
+ tidy_timeout_srcs: ["profile_assistant_test.cc"],
srcs: ["profile_assistant_test.cc"],
}
diff --git a/runtime/Android.bp b/runtime/Android.bp
index 9f8cf68..795ea04 100644
--- a/runtime/Android.bp
+++ b/runtime/Android.bp
@@ -668,6 +668,9 @@
art_cc_defaults {
name: "libart-runtime-gtest-defaults",
+ tidy_timeout_srcs: [
+ "common_runtime_test.cc",
+ ],
srcs: [
"common_runtime_test.cc",
"dexopt_test.cc",
@@ -746,6 +749,23 @@
":art-gtest-jars-VerifierDepsMulti",
":art-gtest-jars-XandY",
],
+ tidy_timeout_srcs: [
+ "arch/stub_test.cc",
+ "class_linker_test.cc",
+ "class_loader_context_test.cc",
+ "hidden_api_test.cc",
+ "instrumentation_test.cc",
+ "interpreter/unstarted_runtime_test.cc",
+ "jni/jni_internal_test.cc",
+ "method_handles_test.cc",
+ "mirror/object_test.cc",
+ "mirror/var_handle_test.cc",
+ "oat_file_assistant_test.cc",
+ "runtime_callbacks_test.cc",
+ "subtype_check_test.cc",
+ "transaction_test.cc",
+ "verifier/reg_type_test.cc",
+ ],
srcs: [
"app_info_test.cc",
"arch/arch_test.cc",
@@ -867,6 +887,9 @@
art_cc_defaults {
name: "art_runtime_compiler_tests_defaults",
+ tidy_timeout_srcs: [
+ "reflection_test.cc",
+ ],
srcs: [
"reflection_test.cc",
],
diff --git a/tools/hiddenapi/Android.bp b/tools/hiddenapi/Android.bp
index 4cd8160..aa7b9f6 100644
--- a/tools/hiddenapi/Android.bp
+++ b/tools/hiddenapi/Android.bp
@@ -83,6 +83,7 @@
":art-gtest-jars-HiddenApi",
":art-gtest-jars-HiddenApiStubs",
],
+ tidy_timeout_srcs: ["hiddenapi_test.cc"],
srcs: ["hiddenapi_test.cc"],
required: ["hiddenapid"],
}