tequilaOS/platform_build_soong
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Fix TestAfdoDeps for darwin

Browse source 
LTO does not set the -import-instr-limit flag when building for
darwin, don't test it on darwin host modules.

Test: afdo_test.go
Change-Id: I83f77006a875f8b0a6af9ae11eb3bad88b0681b1
This commit is contained in:
Colin Cross 2024-02-09 12:19:55 -08:00
parent 15fa814560
commit e00614ecba
1 changed files with 9 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
cc/afdo_test.go
View file

@ -15,6 +15,7 @@
package cc package cc
import ( import (
"runtime"
"strings" "strings"
"testing" "testing"
@ -228,13 +229,15 @@ func TestAfdoDeps(t *testing.T) {
uniqueInternalLinkageNamesCFlag, cFlags) uniqueInternalLinkageNamesCFlag, cFlags)
} }
ldFlags = libTestHost.Rule("ld").Args["ldFlags"] if runtime.GOOS != "darwin" {
ldFlags := libTestHost.Rule("ld").Args["ldFlags"]
if !strings.Contains(ldFlags, noAfdoLtoLdFlag) { if !strings.Contains(ldFlags, noAfdoLtoLdFlag) {
t.Errorf("Expected host 'libTest' to not enable afdo, but did not find %q in ldflags %q", noAfdoLtoLdFlag, ldFlags) t.Errorf("Expected host 'libTest' to not enable afdo, but did not find %q in ldflags %q", noAfdoLtoLdFlag, ldFlags)
} }
if strings.Contains(ldFlags, afdoLtoLdFlag) { if strings.Contains(ldFlags, afdoLtoLdFlag) {
t.Errorf("Expected host 'libTest' to not enable afdo, but found %q in ldflags %q", afdoLtoLdFlag, ldFlags) t.Errorf("Expected host 'libTest' to not enable afdo, but found %q in ldflags %q", afdoLtoLdFlag, ldFlags)
} }
}
// Check dependency edge from afdo-enabled module to static deps // Check dependency edge from afdo-enabled module to static deps
if !hasDirectDep(result, libTestHost.Module(), libFooHost.Module()) { if !hasDirectDep(result, libTestHost.Module(), libFooHost.Module()) {