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

Merge "rust: Emit -x c++ for bindgen modules with cpp_std" into main

Browse source
This commit is contained in:
Ivan Lozano 2023-10-10 13:05:02 +00:00 committed by Gerrit Code Review
commit e94f26e2e8
2 changed files with 16 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
rust/bindgen.go
View file

@ -124,18 +124,20 @@ func (b *bindgenDecorator) getStdVersion(ctx ModuleContext, src android.Path) (s
ctx.PropertyErrorf("c_std", "c_std and cpp_std cannot both be defined at the same time.")
}
if String(b.ClangProperties.Cpp_std) != "" {
if b.ClangProperties.Cpp_std != nil {
isCpp = true
if String(b.ClangProperties.Cpp_std) == "experimental" {
stdVersion = cc_config.ExperimentalCppStdVersion
} else if String(b.ClangProperties.Cpp_std) == "default" {
} else if String(b.ClangProperties.Cpp_std) == "default" || String(b.ClangProperties.Cpp_std) == "" {
stdVersion = cc_config.CppStdVersion
} else {
stdVersion = String(b.ClangProperties.Cpp_std)
}
} else if b.ClangProperties.C_std != nil {
isCpp = false
if String(b.ClangProperties.C_std) == "experimental" {
stdVersion = cc_config.ExperimentalCStdVersion
} else if String(b.ClangProperties.C_std) == "default" {
} else if String(b.ClangProperties.C_std) == "default" || String(b.ClangProperties.C_std) == "" {
stdVersion = cc_config.CStdVersion
} else {
stdVersion = String(b.ClangProperties.C_std)

12
rust/bindgen_test.go
View file

@ -115,7 +115,7 @@ func TestRustBindgenStdVersions(t *testing.T) {
ctx := testRust(t, `
rust_bindgen {
name: "libbindgen_cstd",
wrapper_src: "src/any.h",
wrapper_src: "src/any.hpp",
crate_name: "bindgen",
stem: "libbindgen",
source_stem: "bindings",
@ -141,6 +141,16 @@ func TestRustBindgenStdVersions(t *testing.T) {
if !strings.Contains(libbindgen_cppstd.Args["cflags"], "-std=foo") {
t.Errorf("cpp_std value not passed in to rust_bindgen as a clang flag")
}
// Make sure specifying cpp_std emits the '-x c++' flag
if !strings.Contains(libbindgen_cppstd.Args["cflags"], "-x c++") {
t.Errorf("Setting cpp_std should cause the '-x c++' flag to be emitted")
}
// Make sure specifying c_std omits the '-x c++' flag
if strings.Contains(libbindgen_cstd.Args["cflags"], "-x c++") {
t.Errorf("Setting c_std should not cause the '-x c++' flag to be emitted")
}
}
func TestBindgenDisallowedFlags(t *testing.T) {