diff options
author | 2020-10-28 23:38:33 +0000 | |
---|---|---|
committer | 2020-10-29 00:26:24 +0000 | |
commit | 59e0c7a98f5cabf138db57a2dbe768b8a77cac5e (patch) | |
tree | e3ec89be223af4dd52f9dea10650dd31d47cb1ac /sdk/cc_sdk_test.go | |
parent | 86836489814ac78e195481ae36f137bb98ec4449 (diff) |
Disable sanitizer flags and output files for sanitizer-enabled SDK members.
Test: build/soong/soong_ui.bash --make-mode \
TARGET_PRODUCT=aosp_blueline_hwasan \
TARGET_BUILD_VARIANT=userdebug nothing
Test: build/soong/scripts/build-aml-prebuilts.sh \
runtime-module-{sdk,host-exports} \
platform-mainline-{sdk,test-exports}
Check that generated Android.bp files don't differ
Bug: 171754462
Change-Id: Ieef40ff202ceda2d9e613a5d9295f5c9d8160626
Diffstat (limited to 'sdk/cc_sdk_test.go')
-rw-r--r-- | sdk/cc_sdk_test.go | 77 |
1 files changed, 73 insertions, 4 deletions
diff --git a/sdk/cc_sdk_test.go b/sdk/cc_sdk_test.go index c1813ecd0..84e4f285f 100644 --- a/sdk/cc_sdk_test.go +++ b/sdk/cc_sdk_test.go @@ -464,7 +464,6 @@ func TestSnapshotWithCcSharedLibraryCommonProperties(t *testing.T) { arm64: { export_system_include_dirs: ["arm64/include"], sanitize: { - hwaddress: true, integer_overflow: false, }, }, @@ -496,7 +495,6 @@ cc_prebuilt_library_shared { srcs: ["arm64/lib/mynativelib.so"], export_system_include_dirs: ["arm64/include/arm64/include"], sanitize: { - hwaddress: true, integer_overflow: false, }, }, @@ -527,7 +525,6 @@ cc_prebuilt_library_shared { srcs: ["arm64/lib/mynativelib.so"], export_system_include_dirs: ["arm64/include/arm64/include"], sanitize: { - hwaddress: true, integer_overflow: false, }, }, @@ -548,7 +545,7 @@ sdk_snapshot { `), checkAllCopyRules(` include/Test.h -> include/include/Test.h -.intermediates/mynativelib/android_arm64_armv8-a_shared_hwasan/mynativelib.so -> arm64/lib/mynativelib.so +.intermediates/mynativelib/android_arm64_armv8-a_shared/mynativelib.so -> arm64/lib/mynativelib.so arm64/include/Arm64Test.h -> arm64/include/arm64/include/Arm64Test.h .intermediates/mynativelib/android_arm_armv7-a-neon_shared/mynativelib.so -> arm/lib/mynativelib.so`), ) @@ -2727,3 +2724,75 @@ sdk_snapshot { `), ) } + +func TestNoSanitizerMembers(t *testing.T) { + result := testSdkWithCc(t, ` + sdk { + name: "mysdk", + native_shared_libs: ["mynativelib"], + } + + cc_library_shared { + name: "mynativelib", + srcs: ["Test.cpp"], + export_include_dirs: ["include"], + arch: { + arm64: { + export_system_include_dirs: ["arm64/include"], + sanitize: { + hwaddress: true, + }, + }, + }, + } + `) + + result.CheckSnapshot("mysdk", "", + checkAndroidBpContents(` +// This is auto-generated. DO NOT EDIT. + +cc_prebuilt_library_shared { + name: "mysdk_mynativelib@current", + sdk_member_name: "mynativelib", + visibility: ["//visibility:public"], + installable: false, + compile_multilib: "both", + export_include_dirs: ["include/include"], + arch: { + arm64: { + export_system_include_dirs: ["arm64/include/arm64/include"], + }, + arm: { + srcs: ["arm/lib/mynativelib.so"], + }, + }, +} + +cc_prebuilt_library_shared { + name: "mynativelib", + prefer: false, + visibility: ["//visibility:public"], + compile_multilib: "both", + export_include_dirs: ["include/include"], + arch: { + arm64: { + export_system_include_dirs: ["arm64/include/arm64/include"], + }, + arm: { + srcs: ["arm/lib/mynativelib.so"], + }, + }, +} + +sdk_snapshot { + name: "mysdk@current", + visibility: ["//visibility:public"], + native_shared_libs: ["mysdk_mynativelib@current"], +} +`), + checkAllCopyRules(` +include/Test.h -> include/include/Test.h +arm64/include/Arm64Test.h -> arm64/include/arm64/include/Arm64Test.h +.intermediates/mynativelib/android_arm_armv7-a-neon_shared/mynativelib.so -> arm/lib/mynativelib.so`), + ) +} |