StrixOS/build
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge "Define CLANG_TIDY_UNKNOWN_CFLAGS in tidy.mk"

Browse Source 
am: 8668e9228a

Change-Id: Ifaf2817f1596ecbf22e4cf363dc5098add91c7b1
This commit is contained in:
Chih-hung Hsieh
2018-08-23 15:34:45 -07:00
committed by android-build-merger
parent 8697a7258f 8668e9228a
commit ad3cb5a924
2 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

13
core/clang/tidy.mk
View File

@@ -14,6 +14,19 @@
# limitations under the License. # limitations under the License.
# #
# clang-tidy doesn't recognize every flag that clang does. This is unlikely to
# be a complete list, but we can populate this with the ones we know to avoid
# issues with clang-diagnostic-unused-command-line-argument.
# b/111885396: -flto affected header include directory;
# -fsanitize and -fwhole-program-vtables need -flto.
CLANG_TIDY_UNKNOWN_CFLAGS := \
-Wa,% \
-flto \
-flto=% \
-fsanitize=% \
-fsanitize-% \
-fwhole-program-vtables \
# Returns 2nd word of $(1) if $(2) has prefix of the 1st word of $(1). # Returns 2nd word of $(1) if $(2) has prefix of the 1st word of $(1).
define find_default_local_tidy_check2 define find_default_local_tidy_check2
$(if $(filter $(word 1,$(1))%,$(2)/),$(word 2,$(1))) $(if $(filter $(word 1,$(1))%,$(2)/),$(word 2,$(1)))

1
core/definitions.mk
View File

@@ -105,7 +105,6 @@ ALL_VINTF_MANIFEST_FRAGMENTS_LIST:=
# All tests that should be skipped in presubmit check. # All tests that should be skipped in presubmit check.
ALL_DISABLED_PRESUBMIT_TESTS := ALL_DISABLED_PRESUBMIT_TESTS :=
# CLANG_TIDY_UNKNOWN_CFLAGS is generated by build/soong.
sanitize_tidy_cflags = $(filter-out $(CLANG_TIDY_UNKNOWN_CFLAGS),$1) sanitize_tidy_cflags = $(filter-out $(CLANG_TIDY_UNKNOWN_CFLAGS),$1)
########################################################### ###########################################################