Merge "Allow cfi enabled modules in mixed build." am: d31098e6c2

Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2605807

Change-Id: Ib3baa054cd3b036386916846f88d9a7834a248a9
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Yu Liu
2023-06-06 18:01:53 +00:00
committed by Automerger Merge Worker
2 changed files with 3 additions and 2 deletions

View File

@@ -1928,7 +1928,6 @@ func allEnabledSanitizersSupportedByBazel(c *Module) bool {
unsupportedSanitizers := []*bool{
sanitizeProps.Safestack,
sanitizeProps.Cfi,
sanitizeProps.Scudo,
BoolPtr(len(c.sanitize.Properties.Sanitize.Recover) > 0),
BoolPtr(c.sanitize.Properties.Sanitize.Blocklist != nil),
@@ -1947,6 +1946,8 @@ func allEnabledSanitizersSupportedByBazel(c *Module) bool {
if ubsanEnabled && !c.MinimalRuntimeNeeded() {
return false
}
} else if san == cfi {
continue
} else if c.sanitize.isSanitizerEnabled(san) {
return false
}

View File

@@ -5538,7 +5538,7 @@ func TestDisableSanitizerVariantsInMixedBuilds(t *testing.T) {
name: "foo_cfi",
variant: "android_arm64_armv8-a_static_cfi_apex28",
expectedOutputPaths: []string{
"out/soong/.intermediates/foo_cfi/android_arm64_armv8-a_static_cfi_apex28/foo_cfi.a",
"outputbase/execroot/__main__/foo_cfi.a",
},
},
{