Commit Graph

8875 Commits

Author SHA1 Message Date
Aditya Kumar
2c9b95000e Merge "Remove -mo-strict-align as it is enabled by default in clang-r522817" into main 2024-07-03 00:57:58 +00:00
Tomasz Wasilczyk
41dfb90ed0 Merge "cc_cmake_snapshot: Allow selecting modules from system and vendor" into main 2024-07-02 22:01:19 +00:00
Treehugger Robot
66b451b8ae Merge "AFDO: Do not mutate on disabled modules" into main 2024-07-02 18:00:57 +00:00
Tomasz Wasilczyk
d38d14056c cc_cmake_snapshot: Allow selecting modules from system and vendor
Bug: 285204695
Test: m binder_sdk
Change-Id: I47836dcfa74d48f74e3e7cf0293a8eeb9206be4c
2024-07-02 10:43:02 -07:00
Tomasz Wasilczyk
cd674738c0 cc_cmake_snapshot: refine link and compile flags
Also, move some AIDL detection logic to the AIDL library extension

Bug: 341997808
Test: m binder_sdk
Change-Id: I7776d2a967f547b305b0fa3a68dbe8cba5111534
2024-07-02 10:43:02 -07:00
Yi Kong
f885689227 AFDO: Do not mutate on disabled modules
If a module is implicitly disabled due to compile_multilib: "64" on a
32-bit only device, we would insert a dependency to a non-existant AFDO
profile variation.

Test: presubmit
Bug: 350641338
Change-Id: I82851c4bc943405e48f7f35b3484fac2e509bfe5
2024-07-02 17:07:02 +00:00
Tomasz Wasilczyk
7cec7e6753 Ignore musl libc depencencies
Bug: 349070246
Test: TARGET_SKIP_OTA_PACKAGE="true" TARGET_RELEASE=trunk_staging ./art/tools/build-linux-x86-host-tools.sh
Change-Id: I5393cf33d84eb406399378119ed8734dd7d8c37f
2024-07-01 11:33:25 -07:00
Ivan Lozano
a8a6f549ee Merge "cc/rust: Alias ffi rlib variant for static_libs" into main 2024-06-28 14:44:49 +00:00
Treehugger Robot
fb5251d50d Merge "disable rtti by default even for non-bionic builds" into main 2024-06-27 22:59:08 +00:00
Treehugger Robot
c78f893938 Merge "cc_cmake_snapshot: don't hardcode for linux_glibc:x86_64 hosts" into main 2024-06-27 20:37:01 +00:00
Tomasz Wasilczyk
9288b512ce cc_cmake_snapshot: don't hardcode for linux_glibc:x86_64 hosts
Bug: 349070246
Test: m binder_sdk
Change-Id: Ia1083159d4585d5a699d62face8026b0d49df887
2024-06-27 09:28:56 -07:00
Aditya Kumar
189d5c8283 Remove -mllvm -jump-is-expensive=false from riscv64_device.go
Latest toolchain has this by default.

Change-Id: I920146aee8d5512377bae81b2cba77b7f56b040a
Related: https://github.com/google/android-riscv64/issues/124
Bug: 330200010
2024-06-26 18:55:22 +00:00
Aditya Kumar
e21f1f58df Remove -mo-strict-align as it is enabled by default in clang-r522817
Cherry-picked in aosp/3009313
Related: https://github.com/google/android-riscv64/issues/111
Bug: 327307773
Change-Id: I996378fad09ce50fb432e1b4506ed273e3ec7019
2024-06-26 18:50:23 +00:00
Colin Cross
e5ef3e0ba0 Merge "Make cmake_snapshot output reproducible" into main 2024-06-26 18:09:04 +00:00
Ivan Lozano
fd47b1ab6a cc/rust: Alias ffi rlib variant for static_libs
Alias the rlib variant to "link: static". This allows declaring
rust_ffi_rlib modules in static_libs. This effectively removes any
distinction between rust_ffi_static and rust_ffi_rlib. Removing the
functionality for building Rust staticlib modules will be cleaned up in
a follow-on CL.

This should have the effect of changing the default linkage for all rust
modules in static_libs from linking individual staticlibs to building a
single staticlib that includes all rust_ffi rlib dependencies.

This removes the static_rlibs property, as we're now handling
the choice dynamically. This also makes rlibs only propagate through
cc_library_static modules if the rlib is included in
whole_static_lib. This both mirrors the expected behavior of
cc libraries and helps control which version of a crate ends up in the
final link (e.g. libdoh_ffi vs libdoh_ffi_for_test).

Bug: 254469782
Test: m
Test: m blueprint_tests
Change-Id: I2925f67f6dc9329dae3dcccafb8560900ac8a6fc
2024-06-26 01:35:41 +00:00
Colin Cross
7de1db7f2c Make cmake_snapshot output reproducible
Sort the outputs from cmake_snapshot to make the build.ninja file
reproducible.

Test: cmake_snapshot_test.go
Flag: EXEMPT refactor
Change-Id: I621e0dc9da7ad4c94234e0811f64d347024e3295
2024-06-25 22:55:59 +00:00
mrziwang
abdb293492 Update outputFilesForModuleFromProvider
This CL includes following changes:
1. Added the ability to differentiate the cases that module never
   sets OutputFilesProvider and that module sets the provider with
   a nil value.
2. Updated GenerateTaggedDistFiles to use outputFilesForModuleFromProvider.
3. Updated on cc module to use OutputFilesProvider.

Test: CI
Bug: 339477385
Change-Id: Ib5663a947315f6a90a81b7f073cf8dd22fbb1e05
2024-06-25 15:37:09 -07:00
Wei Li
a1aa297518 Export Soong modules and build a database from metadata from Make and Soong.
Bug: 324465531
Test: CIs
Test: m compliance-metadata.db
Change-Id: Ia1c9ab0ae874dd47969555ddbfb93405b57a651f
2024-06-24 17:20:59 -07:00
Jihoon Kang
79b0eef5a1 Merge "Move vendor and product variant generation logic from cc package to android package" into main 2024-06-21 18:16:31 +00:00
Jihoon Kang
47e918450f Move vendor and product variant generation logic from cc package to android package
Although image variation generation logic has moved out of cc package to
the android package, the vendor and product partition variants
generation logic is still specific to cc package. Therefore, in order to
create a product or vendor variant, they have to specified in
`ExtraImageVariants`. In order to avoid such confusing behaviors and
enforce modules to specify product and vendor installation rules, this
change moves the vendor and product variant generation logic to
android.ImageInterface.

Test: m nothing --no-skip-soong-tests && diff contents of out/soong/Android-{product}.mk
Change-Id: I9e14f3739d9dea94167ee6a91e92b2f942055aba
2024-06-21 07:45:09 +00:00
Treehugger Robot
14a77d3a7f Merge "Minor cc_cmake_snapshot touch-ups" into main 2024-06-20 23:58:00 +00:00
Tomasz Wasilczyk
2493fcc7b5 Minor cc_cmake_snapshot touch-ups
- Partial support for arm64 hosts (no prebuilts yet)
- Replace incorrect template function use MATCHES with STREQUAL
- Add more ignored system libs
- No need to explicitly add libc++ prebuilt

Bug: 285204695
Test: m binder_sdk
Change-Id: Id6508cc9f9ee6328c95c2821c325629f84eda138
2024-06-20 15:30:09 -07:00
Elliott Hughes
b3b05ba84d Remove an out of date comment.
libandroid_support no longer exists.

Change-Id: I6e66f3e0fd5c58bff3c91563a33e0afc8614488c
2024-06-20 16:34:19 +00:00
Zi Wang
26faf1b321 Merge "Use OutputFilesProvider on cc_cmake_snapshot" into main 2024-06-20 16:18:16 +00:00
Pechetty Sravani
f80ffa8b86 Merge "Revert "Enable -Wfortify-source"" into main 2024-06-19 13:15:28 +00:00
Pechetty Sravani
55748021b0 Revert "Enable -Wfortify-source"
This reverts commit 25e9e900f1.

Reason for revert: Droidmonitor triggered revert due to build breakage in b/348142863. Will be verifying through ABTD before submission.

Change-Id: Ia18f83e9261e395cf5f76669523bd6f4a5e9ba49
2024-06-19 11:47:27 +00:00
Maciej Żenczykowski
d957a9bd70 Merge "Enable -Wfortify-source" into main 2024-06-19 10:09:21 +00:00
Aditya Kumar
25e9e900f1 Enable -Wfortify-source
This reverts commit 32e35e416e.

Reason for revert: Fixed the failing builds

Change-Id: I919c292e4802a795771c499a830381ca4c761e18
2024-06-18 21:17:12 +00:00
Cole Faust
ecdb25e674 Merge "Make required a configurable property" into main 2024-06-18 18:05:41 +00:00
mrziwang
f95cfa6087 Use OutputFilesProvider on cc_cmake_snapshot
In the context of incremental soong, the output files
inter-module-communication will be through OutputFilesProvider.
The OutputFileProducer interface will be deprecated.

Test: CI
Bug: 339477385
Change-Id: I38d9b2a953296134490a4347ecd024dac2dd4757
2024-06-18 10:11:39 -07:00
Treehugger Robot
d6352efd1a Merge "Use --lto-O0 for optimizeForSize targets" into main 2024-06-17 21:18:00 +00:00
Cole Faust
43ddd08bd2 Make required a configurable property
So that users can use select statements with it.

Fixes: 347605145
Bug: 342006386
Test: m nothing --no-skip-soong-tests
Change-Id: Ica0ca6d1725b000b3748c0293e5a9f9b38ed87f9
2024-06-17 12:32:40 -07:00
Xin Li
f9fc13dd78 Merge "Merge Android 14 QPR3 to AOSP main" into main 2024-06-17 18:51:33 +00:00
Maciej Żenczykowski
32e35e416e Revert "Enable -Wfortify-source"
This reverts commit ca5e3eb407.

Reason for revert: falcon and seahawk build failures on git_main.

Change-Id: Ib48e33dbb1649641223013739bb85934342a067a
2024-06-17 09:55:36 +00:00
Yi Kong
ae870727b5 Use --lto-O0 for optimizeForSize targets
This is another place where we can trade performance for additional
binary size savings.

Test: presubmit
Change-Id: Icf9600ef2ca4092e4adccc0c26465b542b038f7b
2024-06-16 13:05:05 +09:00
Aditya Kumar
ca5e3eb407 Enable -Wfortify-source
This reverts commit 885f2eec3c.

Reason for revert: Fixed the error reported in post-submit

Change-Id: I152494403e6d6937413d0e5d2637d426c2f54009
2024-06-15 12:07:06 +00:00
Pirama Arumuga Nainar
52436e1574 Merge "Revert^4 "Enable full LTO optimization by default"" into main 2024-06-14 21:54:50 +00:00
Cole Faust
7dc42aa7af Merge "Remove debug/release cflags" into main 2024-06-14 21:15:15 +00:00
Xin Li
c2b69f7fad Merge Android 14 QPR3 to AOSP main
Bug: 346855327
Merged-In: I03a00bfb541769e6aa877a184ce7efa2a03d0f59
Change-Id: I1e408cbdaa040643c946fe92a1da4675232fa667
2024-06-14 10:45:54 -07:00
Aleks Todorov
274b3c9521 Merge "cc: Make generated_headers configurable" into main 2024-06-14 17:25:07 +00:00
Aleks Todorov
36187c371e Merge "cc: Make cc_genrule defaultable" into main 2024-06-14 17:17:17 +00:00
Aleks Todorov
93b6dd0036 cc: Make generated_headers configurable
Bug: 325444956
Test: m
Test: m [custom target with select() in generated_headers]
Change-Id: I95c9aa52be39e0f5afb503f0b71bce222961ec92
2024-06-14 16:56:38 +01:00
Treehugger Robot
523108c2f0 Merge "Add support for recent Mac SDK (13, 14)" into main am: 31934a55a8
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/3106701

Change-Id: I5e003df9742b6d3cdb8d6b0118180bb8495ebcfc
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-06-14 09:59:08 +00:00
Treehugger Robot
31934a55a8 Merge "Add support for recent Mac SDK (13, 14)" into main 2024-06-14 09:57:23 +00:00
Treehugger Robot
d3ba5a3f6e Merge "Cleanup ImageInterface.SetImageVariation" into main am: 77858402e4
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/3131759

Change-Id: I4504623d0978356eb7f3b526d01ffb99abd0da12
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-06-14 04:29:12 +00:00
Treehugger Robot
77858402e4 Merge "Cleanup ImageInterface.SetImageVariation" into main 2024-06-14 04:26:52 +00:00
Cole Faust
0f81d85278 Merge "Make the cflags property configurable" into main am: 1b0d92919d
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/3131289

Change-Id: I56760a2763d6d41d82696d0286085c95d09dfa8d
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2024-06-14 03:08:08 +00:00
Jihoon Kang
7583e835f7 Cleanup ImageInterface.SetImageVariation
This change modifies the interface method of
ImageInterface.SetImageVariation so that the image variation is set
directly at the caller image variation module, instead of passing the
pointer to set the image variation.

Test: m nothing
Change-Id: I8eadb5149365530243e19a8cd37eb49d335fbeef
2024-06-13 21:56:23 +00:00
Cole Faust
e96c16a806 Make the cflags property configurable
So that you can use select statements on it. The current modivation
is to convert a module from make to bp with minimal soong config
variable boilerplate.

Bug: 342006386
Bug: 346922064
Test: m nothing --no-skip-soong-tests
Change-Id: I48d5a11ad3aa65c24cc199458634a5fdbeab2f64
2024-06-13 14:51:14 -07:00
Cole Faust
5591460e81 Remove debug/release cflags
These were never fully implemented, release cflags are the same as
regular cflags, and debug cflags are not used at all.

Bug: 346922064
Test: Presubmits
Change-Id: I0e8914e03f3a092bbc816cabf75255c55c3531e4
2024-06-13 10:49:13 -07:00