StrixOS/build_soong
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

soong: Add crt_pad_segment to .so's am: f4ffe0a026

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

Change-Id: I0b5580c9c2fa0a1f00ab5a617a7c45a29c8d9a5d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Kalesh Singh
2024-02-01 21:39:55 +00:00
committed by Automerger Merge Worker
parent 8e58786f0d f4ffe0a026
commit dc0e1130ac
10 changed files with 61 additions and 18 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
apex/apex_test.go
View File

@@ -1455,6 +1455,7 @@ func TestRuntimeApexShouldInstallHwasanIfLibcDependsOnIt(t *testing.T) {
name: "libc",
no_libcrt: true,
nocrt: true,
no_crt_pad_segment: true,
stl: "none",
system_shared_libs: [],
stubs: { versions: ["1"] },
@@ -1469,6 +1470,7 @@ func TestRuntimeApexShouldInstallHwasanIfLibcDependsOnIt(t *testing.T) {
name: "libclang_rt.hwasan",
no_libcrt: true,
nocrt: true,
no_crt_pad_segment: true,
stl: "none",
system_shared_libs: [],
srcs: [""],
@@ -1511,6 +1513,7 @@ func TestRuntimeApexShouldInstallHwasanIfHwaddressSanitized(t *testing.T) {
name: "libc",
no_libcrt: true,
nocrt: true,
no_crt_pad_segment: true,
stl: "none",
system_shared_libs: [],
stubs: { versions: ["1"] },
@@ -1521,6 +1524,7 @@ func TestRuntimeApexShouldInstallHwasanIfHwaddressSanitized(t *testing.T) {
name: "libclang_rt.hwasan",
no_libcrt: true,
nocrt: true,
no_crt_pad_segment: true,
stl: "none",
system_shared_libs: [],
srcs: [""],

2
cc/config/bionic.go
View File

@@ -24,6 +24,7 @@ var (
bionicCrtBeginStaticBinary, bionicCrtEndStaticBinary = []string{"crtbegin_static"}, []string{"crtend_android"}
bionicCrtBeginSharedBinary, bionicCrtEndSharedBinary = []string{"crtbegin_dynamic"}, []string{"crtend_android"}
bionicCrtBeginSharedLibrary, bionicCrtEndSharedLibrary = []string{"crtbegin_so"}, []string{"crtend_so"}
bionicCrtPadSegmentSharedLibrary = []string{"crt_pad_segment"}
)
func (toolchainBionic) Bionic() bool { return true }
@@ -42,3 +43,4 @@ func (toolchainBionic) CrtBeginSharedLibrary() []string { return bionicCrtBeginS
func (toolchainBionic) CrtEndStaticBinary() []string { return bionicCrtEndStaticBinary }
func (toolchainBionic) CrtEndSharedBinary() []string { return bionicCrtEndSharedBinary }
func (toolchainBionic) CrtEndSharedLibrary() []string { return bionicCrtEndSharedLibrary }
func (toolchainBionic) CrtPadSegmentSharedLibrary() []string { return bionicCrtPadSegmentSharedLibrary }

2
cc/config/toolchain.go
View File

@@ -100,6 +100,7 @@ type Toolchain interface {
CrtEndStaticBinary() []string
CrtEndSharedBinary() []string
CrtEndSharedLibrary() []string
CrtPadSegmentSharedLibrary() []string
// DefaultSharedLibraries returns the list of shared libraries that will be added to all
// targets unless they explicitly specify system_shared_libs.
@@ -161,6 +162,7 @@ func (toolchainNoCrt) CrtBeginSharedLibrary() []string { return nil }
func (toolchainNoCrt) CrtEndStaticBinary() []string { return nil }
func (toolchainNoCrt) CrtEndSharedBinary() []string { return nil }
func (toolchainNoCrt) CrtEndSharedLibrary() []string { return nil }
func (toolchainNoCrt) CrtPadSegmentSharedLibrary() []string { return nil }
func (toolchainBase) DefaultSharedLibraries() []string {
return nil

1
cc/config/x86_linux_host.go
View File

@@ -334,6 +334,7 @@ func (toolchainMusl) CrtBeginSharedLibrary() []string { return muslCrtBeginShare
func (toolchainMusl) CrtEndStaticBinary() []string { return muslCrtEndStaticBinary }
func (toolchainMusl) CrtEndSharedBinary() []string { return muslCrtEndSharedBinary }
func (toolchainMusl) CrtEndSharedLibrary() []string { return muslCrtEndSharedLibrary }
func (toolchainMusl) CrtPadSegmentSharedLibrary() []string { return nil }
func (toolchainMusl) DefaultSharedLibraries() []string { return MuslDefaultSharedLibraries }

4
cc/library.go
View File

@@ -974,6 +974,10 @@ func (library *libraryDecorator) linkerDeps(ctx DepsContext, deps Deps) Deps {
if library.baseLinker.Properties.crt() {
deps.CrtBegin = append(deps.CrtBegin, ctx.toolchain().CrtBeginSharedLibrary()...)
deps.CrtEnd = append(deps.CrtEnd, ctx.toolchain().CrtEndSharedLibrary()...)
}
if library.baseLinker.Properties.crtPadSegment() {
deps.CrtEnd = append(deps.CrtEnd, ctx.toolchain().CrtPadSegmentSharedLibrary()...)
}
deps.WholeStaticLibs = append(deps.WholeStaticLibs, library.SharedProperties.Shared.Whole_static_libs...)
deps.StaticLibs = append(deps.StaticLibs, library.SharedProperties.Shared.Static_libs...)

8
cc/linker.go
View File

@@ -91,6 +91,10 @@ type BaseLinkerProperties struct {
// compiling crt or libc.
Nocrt *bool `android:"arch_variant"`
// don't link in crt_pad_segment. This flag is currently only used internal to
// soong for testing and for vndk prebuilt shared libraries.
No_crt_pad_segment *bool `android:"arch_variant"`
// deprecated and ignored because lld makes it unnecessary. See b/189475744.
Group_static_libs *bool `android:"arch_variant"`
@@ -253,6 +257,10 @@ func (blp *BaseLinkerProperties) libCrt() bool {
return blp.No_libcrt == nil || !*blp.No_libcrt
}
func (blp *BaseLinkerProperties) crtPadSegment() bool {
return blp.No_crt_pad_segment == nil || !*blp.No_crt_pad_segment
}
func NewBaseLinker(sanitize *sanitize) *baseLinker {
return &baseLinker{sanitize: sanitize}
}

6
cc/testing.go
View File

@@ -77,6 +77,7 @@ func commonDefaultModules() string {
no_libcrt: true,
sdk_version: "minimum",
nocrt: true,
no_crt_pad_segment: true,
system_shared_libs: [],
stl: "none",
check_elf_files: false,
@@ -384,6 +385,11 @@ func commonDefaultModules() string {
objs: ["crtbrand"],
}
cc_object {
name: "crt_pad_segment",
defaults: ["crt_defaults"],
}
cc_object {
name: "crtbrand",
defaults: ["crt_defaults"],

10
cc/vendor_snapshot_test.go
View File

@@ -341,6 +341,7 @@ func TestVendorSnapshotUse(t *testing.T) {
vendor: true,
nocrt: true,
no_libcrt: true,
no_crt_pad_segment: true,
stl: "none",
system_shared_libs: [],
compile_multilib: "64",
@@ -458,6 +459,7 @@ func TestVendorSnapshotUse(t *testing.T) {
vendor: true,
nocrt: true,
no_libcrt: true,
no_crt_pad_segment: true,
stl: "none",
system_shared_libs: [],
}
@@ -467,6 +469,7 @@ func TestVendorSnapshotUse(t *testing.T) {
vendor: true,
nocrt: true,
no_libcrt: true,
no_crt_pad_segment: true,
stl: "none",
system_shared_libs: [],
shared_libs: ["libvndk", "libvendor_available", "libllndk"],
@@ -487,6 +490,7 @@ func TestVendorSnapshotUse(t *testing.T) {
vendor: true,
nocrt: true,
no_libcrt: true,
no_crt_pad_segment: true,
stl: "none",
system_shared_libs: [],
static_libs: ["libvendor"],
@@ -501,6 +505,7 @@ func TestVendorSnapshotUse(t *testing.T) {
vendor: true,
nocrt: true,
no_libcrt: true,
no_crt_pad_segment: true,
stl: "none",
system_shared_libs: [],
vndk: {
@@ -597,6 +602,7 @@ func TestVendorSnapshotUse(t *testing.T) {
target_arch: "arm64",
compile_multilib: "both",
vendor: true,
no_crt_pad_segment: true,
shared_libs: [
"libvendor_without_snapshot",
"libvendor_available",
@@ -620,6 +626,7 @@ func TestVendorSnapshotUse(t *testing.T) {
target_arch: "arm64",
compile_multilib: "both",
vendor: true,
no_crt_pad_segment: true,
overrides: ["libvendor"],
shared_libs: [
"libvendor_without_snapshot",
@@ -657,6 +664,7 @@ func TestVendorSnapshotUse(t *testing.T) {
target_arch: "arm64",
compile_multilib: "32",
vendor: true,
no_crt_pad_segment: true,
arch: {
arm: {
src: "lib32.so",
@@ -683,6 +691,7 @@ func TestVendorSnapshotUse(t *testing.T) {
target_arch: "arm64",
compile_multilib: "64",
vendor: true,
no_crt_pad_segment: true,
arch: {
arm64: {
src: "lib64.so",
@@ -722,6 +731,7 @@ func TestVendorSnapshotUse(t *testing.T) {
target_arch: "arm64",
compile_multilib: "both",
vendor: true,
no_crt_pad_segment: true,
arch: {
arm64: {
src: "libvendor_available.so",

1
cc/vndk_prebuilt.go
View File

@@ -232,6 +232,7 @@ func vndkPrebuiltSharedLibrary() *Module {
prebuilt.properties.Check_elf_files = BoolPtr(false)
prebuilt.baseLinker.Properties.No_libcrt = BoolPtr(true)
prebuilt.baseLinker.Properties.Nocrt = BoolPtr(true)
prebuilt.baseLinker.Properties.No_crt_pad_segment = BoolPtr(true)
// Prevent default system libs (libc, libm, and libdl) from being linked
if prebuilt.baseLinker.Properties.System_shared_libs == nil {

5
rust/vendor_snapshot_test.go
View File

@@ -553,6 +553,7 @@ func TestVendorSnapshotUse(t *testing.T) {
vendor: true,
nocrt: true,
no_libcrt: true,
no_crt_pad_segment: true,
stl: "none",
system_shared_libs: [],
}
@@ -857,6 +858,7 @@ func TestVendorSnapshotUse(t *testing.T) {
target_arch: "arm64",
compile_multilib: "32",
vendor: true,
no_crt_pad_segment: true,
arch: {
arm: {
src: "lib32.so",
@@ -870,6 +872,7 @@ func TestVendorSnapshotUse(t *testing.T) {
target_arch: "arm64",
compile_multilib: "64",
vendor: true,
no_crt_pad_segment: true,
arch: {
arm64: {
src: "lib64.so",
@@ -882,6 +885,7 @@ func TestVendorSnapshotUse(t *testing.T) {
target_arch: "arm64",
compile_multilib: "64",
vendor: true,
no_crt_pad_segment: true,
arch: {
arm64: {
src: "liblog.so",
@@ -913,6 +917,7 @@ func TestVendorSnapshotUse(t *testing.T) {
target_arch: "arm64",
compile_multilib: "both",
vendor: true,
no_crt_pad_segment: true,
arch: {
arm64: {
src: "libvendor_available.so",