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

Merge "Change bp2build to reflect arm_isa refactor"

Browse source
This commit is contained in:
Trevor Radcliffe 2023-05-16 19:41:08 +00:00 committed by Gerrit Code Review
commit 735013943a
3 changed files with 3 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
bp2build/cc_binary_conversion_test.go
View file

@ -644,10 +644,7 @@ func TestCcBinaryWithInstructionSet(t *testing.T) {
targets: []testBazelTarget{
{"cc_binary", "foo", AttrNameToString{
"features": `select({
"//build/bazel/platforms/arch:arm": [
"arm_isa_arm",
"-arm_isa_thumb",
],
"//build/bazel/platforms/arch:arm": ["arm_isa_arm"],
"//conditions:default": [],
})`,
"local_includes": `["."]`,

5
bp2build/cc_library_conversion_test.go
View file

@ -3221,10 +3221,7 @@ func TestCcLibraryWithInstructionSet(t *testing.T) {
`,
ExpectedBazelTargets: makeCcLibraryTargets("foo", AttrNameToString{
"features": `select({
"//build/bazel/platforms/arch:arm": [
"arm_isa_arm",
"-arm_isa_thumb",
],
"//build/bazel/platforms/arch:arm": ["arm_isa_arm"],
"//conditions:default": [],
})`,
"local_includes": `["."]`,

2
cc/bp2build.go
View file

@ -500,7 +500,7 @@ func (ca *compilerAttributes) bp2buildForAxisAndConfig(ctx android.BazelConversi
instructionSet := proptools.StringDefault(props.Instruction_set, "")
if instructionSet == "arm" {
ca.features.SetSelectValue(axis, config, []string{"arm_isa_arm", "-arm_isa_thumb"})
ca.features.SetSelectValue(axis, config, []string{"arm_isa_arm"})
} else if instructionSet != "" && instructionSet != "thumb" {
ctx.ModuleErrorf("Unknown value for instruction_set: %s", instructionSet)
}