diff options
| author | Jayant Chowdhary <jchowdhary@google.com> | 2018-03-01 19:12:16 -0800 |
|---|---|---|
| committer | Jayant Chowdhary <jchowdhary@google.com> | 2018-03-02 22:13:15 +0000 |
| commit | ea0a2e1928fd2a3ae14e38ee2a92adc4bd58d446 (patch) | |
| tree | 28dde0f865b248790a527b64a104c916a61c0e80 /cc | |
| parent | dcd33b6c29580298a122711c3f5e1384550cacbe (diff) | |
| download | build_soong-ea0a2e1928fd2a3ae14e38ee2a92adc4bd58d446.tar.gz build_soong-ea0a2e1928fd2a3ae14e38ee2a92adc4bd58d446.tar.bz2 build_soong-ea0a2e1928fd2a3ae14e38ee2a92adc4bd58d446.zip | |
Disable abi diffs for sanitized variants of modules.
asan variants can sometimes have extra exported symbols (this is a vndk
extension violation). Since asan variants are only used for testing,
don't do abi diffs for them.
Bug: 66301104
Test: Without the change, libc++.so.lsdump exists for the asan variant;
with the change, it does not.
Test: make -j64.
Change-Id: I61a4c7a3e9aa0028a54ad0ca8715e8c77aebad94
Diffstat (limited to 'cc')
| -rw-r--r-- | cc/cc.go | 7 |
1 files changed, 6 insertions, 1 deletions
@@ -530,7 +530,12 @@ func (ctx *moduleContextImpl) isVndkExt() bool { // Create source abi dumps if the module belongs to the list of VndkLibraries. func (ctx *moduleContextImpl) createVndkSourceAbiDump() bool { - return ctx.ctx.Device() && ((ctx.useVndk() && ctx.isVndk()) || inList(ctx.baseModuleName(), llndkLibraries)) + isUnsanitizedVariant := true + sanitize := ctx.mod.sanitize + if sanitize != nil { + isUnsanitizedVariant = sanitize.isUnsanitizedVariant() + } + return isUnsanitizedVariant && ctx.ctx.Device() && ((ctx.useVndk() && ctx.isVndk()) || inList(ctx.baseModuleName(), llndkLibraries)) } func (ctx *moduleContextImpl) selectedStl() string { |