Commit Graph

66207 Commits

Author SHA1 Message Date
Colin Cross
27da704772 Merge "Add tests for code that will be affected by resource processor feature" into main am: 120b55a7ca
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2649208

Change-Id: I99967322c62a5a770032f826e4cb49a42b96dc07
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-11 20:48:56 +00:00
Colin Cross
120b55a7ca Merge "Add tests for code that will be affected by resource processor feature" into main 2023-07-11 20:17:05 +00:00
Wei Li
1a128321e3 Merge "Add integration test for m build SBOM of APEXs/APKs." into main am: 1ab0564e65
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2652824

Change-Id: Ieaf85e8ea018f834e9a1dc4bf122b03d857a8d9b
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-11 18:41:05 +00:00
Treehugger Robot
cad6091fc8 Merge "Turn on Global ThinLTO by default" into main am: 71f5a63267
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2200436

Change-Id: I4105caa231e8df9d9125d5680a3398028f6852f2
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-11 18:40:52 +00:00
Treehugger Robot
f0fa7b4edf Merge "Experiment with not using ninja weights" into main am: da7973112a
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2652025

Change-Id: I175897787fc69913b390dbba4be6d4e32ab85f70
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-11 18:38:29 +00:00
Wei Li
1ab0564e65 Merge "Add integration test for m build SBOM of APEXs/APKs." into main 2023-07-11 18:25:39 +00:00
Treehugger Robot
71f5a63267 Merge "Turn on Global ThinLTO by default" into main 2023-07-11 17:42:12 +00:00
Treehugger Robot
da7973112a Merge "Experiment with not using ninja weights" into main 2023-07-11 17:42:06 +00:00
Romain Jobredeaux
5484ae074f Merge "Support target and max sdk version in app bp2build." into main am: aa634fdbdb
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2650657

Change-Id: Ia09f51d07aad7c1b22d58b79376ab616f9cf8dd5
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-11 14:13:19 +00:00
Romain Jobredeaux
aa634fdbdb Merge "Support target and max sdk version in app bp2build." into main 2023-07-11 13:34:29 +00:00
Yi Kong
add6375a10 Turn on Global ThinLTO by default
Build C/C++ targets with ThinLTO with "--lto-O0" by default. This takes
advantage of better dead code elimination and CFG simplification due to
ThinLTO's global view of the program, but do not enable the costly LTO
optimizations. This also makes builds faster because we can avoid doing
codegen for the dead code.

Code size for panther-userdebug:
                   /system/bin  /system/lib  /system/lib64
Original           57648        86264        181700
Global ThinLTO     55764        84916        175728
                   -3.27%       -1.56%       -3.29%

Build time for aosp_arm64-userdebug:
                   clang time   linker time
Original           56993.87s    1712.36s
Global ThinLTO     52839.18s    3114.93s

saving 4.68% build time in total.

Bug: 169004486
Test: presubmit
Change-Id: Idb0f4675ca4750b8b12b24f4679579419d3448c8
2023-07-11 21:26:52 +09:00
Aditya Kumar
682eb3468e Merge "gsimple-template-names" into main am: 9488eb82d4
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2607728

Change-Id: I4a504cd5fed09f7d7e3d3aa5f15f51adb9c8822b
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-11 06:29:01 +00:00
Aditya Kumar
9488eb82d4 Merge "gsimple-template-names" into main 2023-07-11 05:54:11 +00:00
Treehugger Robot
84d32c5b1c Merge "Copy imported APKs to output filename" into main am: 5af242249d
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2652823

Change-Id: I087856e45b24e9a0fadceecca1c0dcb13cb9ef81
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-11 05:26:46 +00:00
Treehugger Robot
5af242249d Merge "Copy imported APKs to output filename" into main 2023-07-11 04:52:46 +00:00
Igor Foox
1113811927 Merge "Update genrule denylist to make m droid build." into main am: 921a465919
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2651202

Change-Id: Ic15817a72cdebb304f115499171c74027e4c0573
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-10 23:28:55 +00:00
Igor Foox
921a465919 Merge "Update genrule denylist to make m droid build." into main 2023-07-10 22:51:44 +00:00
Wei Li
59df0ee442 Add integration test for m build SBOM of APEXs/APKs.
Bug: 266726655
Test: build/soong/tests/sbom_test.sh
Change-Id: Id8002042f607f8e158f561ca47e0a84a8b8927e2
2023-07-10 15:50:38 -07:00
Colin Cross
5780d57a71 Copy imported APKs to output filename
android_app_import and android_test_import modules with processed: true
and skip_preprocessed_apk_checks are used directly from the source tree.
If the source file name doesn t match the module name and the file is
used as test data it can result in the wrong filename being used.  Copy
the source file to an output file with the correct name first.

Bug: 290376750
Test: m CtsAppSecurityHostTestCases
Change-Id: I4a6dd1c5e48db7085ea41035def31f0844948a46
2023-07-10 22:18:01 +00:00
Igor Foox
04ff5f90ae Update genrule denylist to make m droid build.
Test: lunch aosp_cf_x86_phone && GENRULE_SANDBOXING=true m droid
Change-Id: I9092df9a407ffe9a7f1c12f18fb0ce1c5fa027a1
2023-07-10 14:58:51 -04:00
AdityaK
423e4ce9e3 gsimple-template-names
Based on: https://chromium-review.googlesource.com/c/chromium/src/+/3988987

```
Savings are almost 330M on 11G

du -s out.diff/target/product/vsoc_arm64/symbols
10751692        out.diff/target/product/vsoc_arm64/symbols
du -s out/target/product/vsoc_arm64/symbols
11086508        out/target/product/vsoc_arm64/symbols
```

Change-Id: I49a5c4cadc77cb0d97e680e95c4efa7156a42f13
2023-07-10 17:36:22 +00:00
Romain Jobredeaux
496660d38a Support target and max sdk version in app bp2build.
Test: Unit tests + CI
Bug: 285304294
Bug: 280465047
Change-Id: I9bb1b6f9729ad783a43837d77dcf2d02b50ca77e
2023-07-10 11:50:00 -04:00
Elliott Hughes
4f31fc623d Merge "Remove -fstrict-aliasing from arm32." into main am: ff96fe1589
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2646182

Change-Id: I6be82a82dbb7161d78ceb702eddececab01e245a
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-10 15:20:10 +00:00
Elliott Hughes
ff96fe1589 Merge "Remove -fstrict-aliasing from arm32." into main 2023-07-10 14:37:39 +00:00
Treehugger Robot
6e57e9b38f Merge "kzip build: pipe error message to stderr." into main am: ab8ed75b6b
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2648202

Change-Id: I289fa5f946fcfb3134b49afa9a11255d93af1344
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-10 12:31:30 +00:00
Treehugger Robot
ab8ed75b6b Merge "kzip build: pipe error message to stderr." into main 2023-07-10 11:45:09 +00:00
Jingwen Chen
ce08efd395 kzip build: pipe error message to stderr.
This was going to stdout so it wasn't showing up on build_error.log.

Test: n/a
Change-Id: I58ac08961b089d06720a307ece3398594b3456de
2023-07-10 08:18:58 +00:00
Treehugger Robot
ed7ff79589 Merge "Implement OutputFileProducer for android_app_import and android_test_import" into main am: b2ace8d72d
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2652340

Change-Id: Ifbcda5bd7ca654591b0c6acde5ca0c60f689d1c3
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-08 01:18:52 +00:00
Treehugger Robot
b2ace8d72d Merge "Implement OutputFileProducer for android_app_import and android_test_import" into main 2023-07-08 00:36:53 +00:00
Treehugger Robot
3f7fb697db Merge "rust: Allow any host tool for custom_bindgen" into main am: 00042a4875
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2651301

Change-Id: I650b298f0325f1774742aea4890cce6d07f52698
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-07 22:51:47 +00:00
Liz Kammer
8822ff5193 Experiment with not using ninja weights
Test: CI
Change-Id: Id611059dba35aaaebc7cf787381669c4294ceb53
2023-07-07 18:31:39 -04:00
Treehugger Robot
00042a4875 Merge "rust: Allow any host tool for custom_bindgen" into main 2023-07-07 22:10:25 +00:00
Jason Macnak
87d6e348da Merge "Remove FrontendStub_[cc|h] from SandboxingDenyModuleList" into main am: 4ae9c14557
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2631752

Change-Id: I944f14abfd17d4f7bb035ce5919992cac5362a4f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-07 20:03:57 +00:00
Elliott Hughes
9bbe96068d Remove -fstrict-aliasing from arm32.
Our global default is -fno-strict-aliasing (including for 32-bit x86),
so let's avoid a source of arm32-only bugs.

Bug: http://b/31600139
Test: treehugger
Change-Id: Iaf3069200b2969718593387dc5b756c574a5d68a
2023-07-07 19:45:24 +00:00
Jason Macnak
4ae9c14557 Merge "Remove FrontendStub_[cc|h] from SandboxingDenyModuleList" into main 2023-07-07 19:16:14 +00:00
Colin Cross
5368d0b44a Implement OutputFileProducer for android_app_import and android_test_import
Fixes: 290376750
Test: builds
Change-Id: I5fc3a759462cfd4342b9330ac8978368feee26ca
2023-07-07 18:36:08 +00:00
Colin Cross
ff3ff7f354 Add tests for code that will be affected by resource processor feature
Add a test for the aapt2 resource processing code that will be affected
by the resource processor feature in a subsequent CL.

Test: TestAndroidResourceProcessor
Change-Id: I39712853f7bbfb7963f1b71e234d039d5ccaae53
2023-07-07 10:58:31 -07:00
Colin Cross
14bb24da15 Merge changes I70f81120,If8f2534e into main am: 702ef0a07a
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2650576

Change-Id: Icef910dca4e3e239db6550023fc252794da4a8c3
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-07 17:37:33 +00:00
Colin Cross
702ef0a07a Merge changes I70f81120,If8f2534e into main
* changes:
  Export ReverseSliceInPlace and ReverseSlice
  Make CopyOf generic
2023-07-07 17:07:50 +00:00
Andrei Homescu
4494685aec rust: Allow any host tool for custom_bindgen
Switch custom_bindgen to use any HostToolProvider
instead of just Rust modules, so any host binary can
be used as the bindgen wrapper.

Bug: 242243245
Test: presubmit
Change-Id: I9ba189b2857b3049dcf14842c178ce5846bab2f7
2023-07-07 04:59:07 +00:00
Yabin Cui
e4eb12c8fa Merge "Update clang version to clang-r498229" into main am: f67bc44848
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2618222

Change-Id: Ic47f171e1a2f54aae461bb0be290513f2bdc3a8f
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-06 23:58:19 +00:00
Cole Faust
d2b95df46c Merge "Handle RepoMappingManifest actions" into main am: 39381dc2bd
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2634063

Change-Id: I5c79d2a14223c138c61bb5abb08825a6411484b2
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-06 23:57:32 +00:00
Yabin Cui
f67bc44848 Merge "Update clang version to clang-r498229" into main 2023-07-06 23:54:42 +00:00
Colin Cross
b5e3f7dad5 Export ReverseSliceInPlace and ReverseSlice
... and add tests.

Test: TestReverseSliceInPlace, TestReverseSlice
Change-Id: I70f811207fdb2af891f7ef3fe19cbe5c7276135a
2023-07-06 16:36:50 -07:00
Cole Faust
39381dc2bd Merge "Handle RepoMappingManifest actions" into main 2023-07-06 23:14:11 +00:00
Colin Cross
13aeb689af Make CopyOf generic
Test: util_test.go
Change-Id: If8f2534ec454903eb967dc9ec0c8708096d1baff
2023-07-06 15:02:56 -07:00
Cole Faust
950689a9c2 Handle RepoMappingManifest actions
These are essentially FileWrite actions, like SourceSymlinkManifest.

However, currently aquery appears to give non-deterministic results
for the file content. Emit an empty file for now to avoid the
non-determinism.

Fixes: 288292933
Bug: 287539324
Test: Presubmits
Change-Id: Ieaf8aff0d58d2d69087e4e34dc99074a015e700c
2023-07-06 09:44:14 -07:00
Treehugger Robot
df2cf568a8 Merge "Revert "Commend out dcla_apex_comparison_test.sh"" am: a9778d576b
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2646204

Change-Id: If9ccd9d9617244a1988257fbd45dc598fa0689ed
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-06 02:05:36 +00:00
Treehugger Robot
a9778d576b Merge "Revert "Commend out dcla_apex_comparison_test.sh"" 2023-07-06 01:27:04 +00:00
Treehugger Robot
e568082a9a Merge "Add comment for afdo flag" am: 253054a175
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2640130

Change-Id: I08fcca2524cf822d9c895ee6000d2c4d9d16e264
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
2023-07-05 21:29:18 +00:00