Commit graph

7079 commits

Author SHA1 Message Date
Treehugger Robot
ce2d8950a0 Merge "Memory mapped coverage (take 2)" am: c8a6487153 am: 079c381504 am: 4a759d5b33
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2009378

Change-Id: I29ebf63aad871a4c569d399ef54e69ebc459f117
2022-03-08 07:54:54 +00:00
Treehugger Robot
4a759d5b33 Merge "Memory mapped coverage (take 2)" am: c8a6487153 am: 079c381504
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2009378

Change-Id: If280e01532aef4a703d0ffc3038ef4773526f8ef
2022-03-08 07:31:50 +00:00
Pirama Arumuga Nainar
b37ae58a3d Memory mapped coverage (take 2)
Add a build variable, CLANG_COVERAGE_CONTINUOUS_MODE, instead of
selecting based on paths.

Test: CLANG_COVERAGE_CONTINUOUS_MODE=true m and verify continuous mode works

Change-Id: I731172fc1f00e1cabff8efcf8b99f9a48210b446
2022-03-04 14:07:53 -08:00
Liz Kammer
bdd24b1ea3 Merge "Export variant/features via soong_injection" am: 6e4cd27bec am: a1b33112b1 am: 1bf0a42c98
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2000396

Change-Id: I361123ce6d7c1dc4400fac67b13373c022a800b8
2022-03-03 19:22:59 +00:00
Liz Kammer
1bf0a42c98 Merge "Export variant/features via soong_injection" am: 6e4cd27bec am: a1b33112b1
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2000396

Change-Id: I6b403e30a36113a4bda62eef6d94f58c5f79e9ed
2022-03-03 18:59:55 +00:00
Liz Kammer
6e4cd27bec Merge "Export variant/features via soong_injection" 2022-03-03 18:11:25 +00:00
Treehugger Robot
787bf8a5f4 Merge "Revert "[cc_fuzz] Revert 'disable LTO' patches."" am: 5c8693f5d7 am: 3807ca1cdf am: eafe37dfb7
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2004383

Change-Id: I3d5f6c2a9e539d0a9942d5c50471987dc4e887d7
2022-03-02 06:29:00 +00:00
Treehugger Robot
eafe37dfb7 Merge "Revert "[cc_fuzz] Revert 'disable LTO' patches."" am: 5c8693f5d7 am: 3807ca1cdf
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/2004383

Change-Id: Ifa0e8a069b09edc4ba9a6988306291c1df5fd4de
2022-03-02 06:05:40 +00:00
Treehugger Robot
5c8693f5d7 Merge "Revert "[cc_fuzz] Revert 'disable LTO' patches."" 2022-03-02 05:08:27 +00:00
Mitch Phillips
5007c4a06d Revert "[cc_fuzz] Revert 'disable LTO' patches."
Revert submission 1976512-revert-nolto-fuzzing

Reason for revert: b/222160662
Reverted Changes:
Iacee4fa29:[cc_fuzz] Revert 'disable LTO' patches.
Ic509b00a1:[cc_fuzz] Revert 'disable LTO' patches.
If85931f09:[cc_fuzz] Revert 'disable LTO' patches.

Change-Id: I84e01a4f84145e3331c2955836c8cc9bfa05c36b
2022-03-02 01:25:22 +00:00
Treehugger Robot
0e97819e91 Merge "[cc_fuzz] Revert 'disable LTO' patches." am: 6c0de9938e am: 998e34c8fd am: 29ad0112d5
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1976512

Change-Id: I77603015a36ac8583d0ea602bd37b57f5ef575bb
2022-03-02 00:55:17 +00:00
Treehugger Robot
29ad0112d5 Merge "[cc_fuzz] Revert 'disable LTO' patches." am: 6c0de9938e am: 998e34c8fd
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1976512

Change-Id: Iec08cb88461a4502c94b7d58cd4749b4fc9d1ad1
2022-03-02 00:33:57 +00:00
Treehugger Robot
6c0de9938e Merge "[cc_fuzz] Revert 'disable LTO' patches." 2022-03-01 23:43:04 +00:00
Colin Cross
655be5dba9 Merge "Use single module for clang runtime libraries" into sc-v2-dev-plus-aosp am: 135513ae65
Original change: https://googleplex-android-review.googlesource.com/c/platform/build/soong/+/17036885

Change-Id: I72d5cdd315cad007f6f6a1a1a91f8ca272014f9c
2022-03-01 01:20:47 +00:00
Colin Cross
fc0df95df5 Use single module for clang runtime libraries
The clang prebuilts now provide a single module with per-architecture
variants instead of a module per architecture.

Bug: 220019988
Test: m checkbuild
Change-Id: I39e2cf8ae14edf8510276dab38011afaef85822c
(cherry picked from commit 386ee59a23)
Merged-In:I39e2cf8ae14edf8510276dab38011afaef85822c
2022-02-28 21:01:05 +00:00
Liz Kammer
e8303bd0ae Export variant/features via soong_injection
Test: build/bazel/ci/bp2build.sh
Change-Id: If70043dc9d020d5f4a243ba4b192b99c9c7e8563
2022-02-28 14:17:15 -05:00
Treehugger Robot
0b182eddff Merge "Fix snapshot symlinks" am: 97491ead9e am: abdcb636cb am: 7783beb7e8
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1998310

Change-Id: If2fa266644667de4d44927f41867dbcc3217fe98
2022-02-25 01:48:59 +00:00
Treehugger Robot
97491ead9e Merge "Fix snapshot symlinks" 2022-02-25 00:41:35 +00:00
Inseob Kim
4d945ee719 Fix snapshot symlinks
LOCAL_MODULE_SYMLINKS was enough to install symlinks for snapshot
binaries. However, Soong now requires LOCAL_SOONG_INSTALLED_SYMLINKS,
which results in broken symlink support in snapshots.

This fixes symlink problems, and then amends the Soong snapshot test.

Bug: 220639435
Bug: 220907049
Test: manually add symlinks and try building
Test: Soong test
Change-Id: Ia75ddf3aa642558e396f1d80736310c3c8d02ad1
2022-02-24 10:29:18 +09:00
Treehugger Robot
415b0d442f Merge "Fix typos in arch_list and do variant validation" am: cda344f120 am: e8815d5ebb am: 94fff59456
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1982666

Change-Id: I9f6be5cd2673b4e5e5eef52ee8132b0c31d348ba
2022-02-23 18:43:29 +00:00
Sam Delmerico
b80790ec41 Merge "convert java proto libraries with bp2build" am: bc83b504ff am: 4dc3df7be7 am: 0d8dd4692b
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1975287

Change-Id: Ie93ce69b956aedbd4999283886ef9340d88fe530
2022-02-23 17:57:13 +00:00
Martin Stjernholm
24424b5678 Mark implementation variant of prebuilts with stubs as not installable am: 5bdf2d589c am: 60da7cf31e am: 8d9cd1becc
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1974259

Change-Id: I5c30bc8038dbb40827cb052e9e454510a7899632
2022-02-23 17:56:45 +00:00
Treehugger Robot
cda344f120 Merge "Fix typos in arch_list and do variant validation" 2022-02-23 17:34:51 +00:00
Sam Delmerico
bc83b504ff Merge "convert java proto libraries with bp2build" 2022-02-23 16:13:15 +00:00
Martin Stjernholm
5bdf2d589c Mark implementation variant of prebuilts with stubs as not installable
This is necessary to avoid installing them. Previously, when install
dependencies were resolved in make, they weren't installed because they
got a .bootstrap suffix in their mk modules. However when that logic
moved into Soong by resolving transitive dependencies in
computeInstallDeps, they started to get dependencies and hence their
stubs could get installed, e.g. system/lib{,64}/libdexfile.so from
prebuilt_libdexfile.

Test: m nothing
Test: env NINJA_ARGS="-t path droid out/target/product/vsoc_x86_64/system/lib64/libdexfile.so" \
        m SOONG_CONFIG_art_module_source_build=false nothing
  verify that ninja reports no dependency path
Bug: 211770050
Bug: 220898484
Change-Id: Ifbfe31a15428926ce57b9e91b535b7ae79038fbd
2022-02-23 09:19:18 +00:00
Yi Kong
91d66d3c04 Merge "Turn on Unique Internal Names for AFDO" am: e69b021758 am: 38d0106dad am: e50f27d358
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1977996

Change-Id: Ic0f538f15c1a23febcb6257a6f4d421028f69ee0
2022-02-22 13:17:07 +00:00
Yi Kong
e69b021758 Merge "Turn on Unique Internal Names for AFDO" 2022-02-22 12:07:55 +00:00
David Anderson
a480cc78ef Merge "Add an __ANDROID_RAMDISK__ preprocessor define." am: 7309742728 am: 6a3b60969e am: 13f34bef8d
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1989090

Change-Id: Ibbd1c1130c90f28aa5822fe73c907d55bfd14df1
2022-02-19 00:56:11 +00:00
David Anderson
7309742728 Merge "Add an __ANDROID_RAMDISK__ preprocessor define." 2022-02-18 23:29:27 +00:00
Treehugger Robot
a7a2b3046b Merge "Add tidy_timeout_srcs property" am: 72518f45e9 am: 01ec0a21d2 am: 730f0b61c9
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1991252

Change-Id: I37fb415d2fcea042c45ea1b722cf0b6c596a2409
2022-02-18 22:09:14 +00:00
Colin Cross
e46bd5363e Merge "Export sanitizer paths to Make" am: 2e91786932 am: 6ef7afc9fe am: 93d5b84636
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1990671

Change-Id: I139bb27c91f857fe6df4778cac08276f90a4ed3d
2022-02-18 22:08:52 +00:00
Treehugger Robot
1b8642069a Merge "Propagate data_bins from Soong to Make" am: 309ee8d398 am: ceb712dfe5 am: 534543e608
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1981828

Change-Id: I1da27920106c36208316d3bd4bcbb375b66b22b0
2022-02-18 22:08:36 +00:00
Yu Liu
d9202b0588 Merge "Enable cc_binary in mixed build." am: 9d7d9810e2 am: ea1aae2889 am: 4d7ec99d78
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1983488

Change-Id: Idfd7f6e90ef4dc78ef6b4ec777c81d0d40a71ecb
2022-02-18 10:39:42 +00:00
Florian Mayer
56ce8e4285 Merge "Reland^2 "Enable hwasan use after scope detection."" am: 939143a862 am: a8c5d3792d am: c516ab982e
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1988614

Change-Id: Ib57f45fbb43fdd1a39536dd9b9024d867abb941b
2022-02-18 10:39:36 +00:00
Chih-hung Hsieh
d3bb913740 Merge "Do not add non-existing -Bprebuilts/gcc/.../bin" am: 70387c5d03 am: 1918a6120a am: a59f62bfdf
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1987029

Change-Id: I6086e708fab979e2f864997e087f2e5dc2e18af6
2022-02-18 10:39:27 +00:00
Pirama Arumuga Nainar
d9f0822a4e Merge "Reland "Switch to clang-r445002"" am: 2b31fbb805 am: c31610cbce am: 640d3f8587
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1985009

Change-Id: I9b8dadac0a0a7da292f3fec72d9035925219abf3
2022-02-18 10:39:18 +00:00
Treehugger Robot
72518f45e9 Merge "Add tidy_timeout_srcs property" 2022-02-18 03:57:01 +00:00
Colin Cross
2e91786932 Merge "Export sanitizer paths to Make" 2022-02-18 03:39:26 +00:00
Treehugger Robot
309ee8d398 Merge "Propagate data_bins from Soong to Make" 2022-02-18 01:26:07 +00:00
Colin Cross
dd5655b3a4 Export sanitizer paths to Make
system/core/rootdir/Android.mk writes a list of sanitizer libraries
to sanitizer.libraries.txt, and assumes that they are installed with
the same name as the module.  The next patch renames the module to
be the same for all architectures while keeping the installed name
as is.  Collect the output file names of the libraries to export
to make.

Bug: 220019988
Test: m out/target/produuct/coral/system/etc/sanitizer.libraries.txt
Change-Id: Idc51c2ad6f914977a286fe4e2fcb457bc1229339
2022-02-17 15:52:07 -08:00
Chih-Hung Hsieh
9db8a0c5b2 Add tidy_timeout_srcs property
Similar to `tidy_disabled_srcs`, a `tidy_timeout_srcs` list
can be used to include all source files that took long to compile
with clang-tidy. Files listed in `tidy_timeout_srcs` will not
be compiled by clang-tidy when `TIDY_TIMEOUT` is defined.

Bug: 201099167
Test: TIDY_TIME=90 make droid tidy-soong_subset
Change-Id: Ie0bfda66caae4445d10117ceefa1b5b8c1ecf256
2022-02-17 14:48:40 -08:00
Yu Liu
9d7d9810e2 Merge "Enable cc_binary in mixed build." 2022-02-17 21:54:04 +00:00
Florian Mayer
939143a862 Merge "Reland^2 "Enable hwasan use after scope detection."" 2022-02-17 21:19:36 +00:00
Florian Mayer
0b981f54e6 Reland^2 "Enable hwasan use after scope detection."
This CL enables HWASan to detect a new class of bugs, specifically
use-after-scope. An example for a bug like this is

int* y;
{
   int x = 1;
   y = &x;
}
*y = 2;

IF YOU FOUND THIS CL AS A POSSIBLE CULPRIT OF A TEST FAILURE:

While it is possible that there is a bug in HWASan and this CL needs
to be rolled back, please also consider that this might surface
actual problems in either the test code or the code under test. See
https://r.android.com/1956922 for an example of fix for a bug
detected by a previous rollout of this flag.

This reverts commit fd337b3963.

Reason for revert: Once https://r.android.com/1985009 is submitted the bug that caused the revert will be fixed.

Change-Id: Id9e81e8b7c26e044af00bdaeae6bb35abbbd9710
2022-02-17 20:52:15 +00:00
Yu Liu
7f3605fee1 Enable cc_binary in mixed build.
Bug: 216114082
Test: Run mixed build on adbd apex and verify the pavkaged adbd binary;
add unit test.
Change-Id: Id11549a7952a7ce5ac02988b88b747bbfe8339ff

Change-Id: If01efe076c223886278f466b65628602bff3a964
2022-02-17 11:15:59 -08:00
Chih-hung Hsieh
70387c5d03 Merge "Do not add non-existing -Bprebuilts/gcc/.../bin" 2022-02-17 18:25:38 +00:00
Pirama Arumuga Nainar
2b31fbb805 Merge "Reland "Switch to clang-r445002"" 2022-02-17 18:07:38 +00:00
David Anderson
2c8075cd6f Add an __ANDROID_RAMDISK__ preprocessor define.
There is no easy way to inform C++ targets whether they're building "not
in normal Android". In the past we used __ANDROID_RECOVERY__, but this
isn't defined for ramdisk targets, or for vendor_boot. Furthermore
there's no "target.ramdisk.cflags" option to add it.

Add a new __ANDROID_RAMDISK__ define that will be set in any ramdisk or
recovery target (since recovery by definition is in a ramdisk).

Bug: 205987817
Test: mm bootimage, vendorbootimage, recovery
Change-Id: If3cd34b07fe0e20abc5108cb338a56698d33e42e
2022-02-16 21:59:05 -08:00
yangbill
22bafec042 Propagate data_bins from Soong to Make
Write the list of a test module's data_bins value to the
`LOCAL_TEST_DATA_BINS` Makefile variable defined for each module.
This enables downstream tools to correctly set up the runtime
environment for execution. And currently only sh_test, cc_tests, and
rust_tests has this attribute in Android.bp.

Bug: 215234071
Test: m out/soong/Android-aosp_cf_x86_64_phone.mk
Change-Id: I8d47f5f9b25afdc5975c6b414405badb38dbde4b
2022-02-17 04:06:34 +00:00
Colin Cross
d42fcae75f Merge "Add --sysroot /dev/null to musl compiles" am: e4c400bef5 am: 69324f735d am: b5e24d257f
Original change: https://android-review.googlesource.com/c/platform/build/soong/+/1983627

Change-Id: Id40f845a799bc1ed8fc565ed9c909946bacbf6d2
2022-02-17 00:40:40 +00:00