Commit graph

5135 commits

Author SHA1 Message Date
Pirama Arumuga Nainar
5f1de0f704 Merge "Add enable_profile_use property" am: 0e8afed83a am: ba6593f5fa am: 4b5fab84eb
am: 062a825a08

Change-Id: I179ae252f43aeeebc61e440a1949222e2a3e8318
2017-10-17 04:07:46 +00:00
Colin Cross
7e7b036bea Refactor errorprone support to reduce duplication am: 070879e69e am: 5e8a9565dd am: 7362317361
am: 36cdbf0ac8

Change-Id: Ic3a6faa8aaed20dab8d09e9f8f90a6546e421236
2017-10-17 04:07:23 +00:00
Pirama Arumuga Nainar
062a825a08 Merge "Add enable_profile_use property" am: 0e8afed83a am: ba6593f5fa
am: 4b5fab84eb

Change-Id: I59b7f2a10911fadbc38d9270fd41e83d8cc28457
2017-10-17 03:35:04 +00:00
Pirama Arumuga Nainar
4b5fab84eb Merge "Add enable_profile_use property" am: 0e8afed83a
am: ba6593f5fa

Change-Id: Iab42162c2bb7616d594e9f76e7c2cdc76a0892a2
2017-10-17 03:19:24 +00:00
Pirama Arumuga Nainar
ba6593f5fa Merge "Add enable_profile_use property"
am: 0e8afed83a

Change-Id: I39b7a8d6ad3d794ed8f20100fa964acb12c02937
2017-10-17 03:07:22 +00:00
Colin Cross
36cdbf0ac8 Refactor errorprone support to reduce duplication am: 070879e69e am: 5e8a9565dd
am: 7362317361

Change-Id: I554d44937b2582040ffe2477f82a5e5967194004
2017-10-17 03:03:01 +00:00
Pirama Arumuga Nainar
0e8afed83a Merge "Add enable_profile_use property" 2017-10-17 03:00:05 +00:00
Colin Cross
7362317361 Refactor errorprone support to reduce duplication am: 070879e69e
am: 5e8a9565dd

Change-Id: I7fd478ca6f5f20ff95f345d3db48ca5adb4b1524
2017-10-17 02:03:48 +00:00
Colin Cross
5e8a9565dd Refactor errorprone support to reduce duplication
am: 070879e69e

Change-Id: I8d06cb4b506e304fcfb2083cb1afc3c8991856a5
2017-10-17 01:59:48 +00:00
Pirama Arumuga Nainar
6aeed8b439 Add enable_profile_use property
Bug: http://b/65598278

This property defaults to 'true' and setting it to false skips profile
use.  This escape hatch lets us disable PGO for a module without
completely removing the 'pgo' property.  Additionally, this also helps
selectively disabling PGO for some architectures, if desired.

Test: Test that -fprofile-use is not added for a test module if
'enable_profile_use: false' is set.

Change-Id: Ifcf1a48c194bc86efd88a529cc2d66a47b7ab080
2017-10-16 13:36:29 -07:00
Colin Cross
070879e69e Refactor errorprone support to reduce duplication
Use a helper function to set up errorprone and javac compiles.

Test: m -j checkbuild
Test: m -j RUN_ERROR_PRONE=true
Change-Id: Icef3a5e1b359487eea3c3306d3d5763dab912b38
2017-10-16 12:50:29 -07:00
Dan Willemsen
f019dc4124 Revert "Always run asan ckati on the build servers" am: 8122bd58ba am: 63f872a239 am: e2cc00a17c
am: 80ea4ef47c

Change-Id: I34858bad5fc2e8b28e25a595c07be5ce7f509cfe
2017-10-16 01:09:47 +00:00
Dan Willemsen
59318ed8f1 Always record kati stats into soong log am: 75d2c170b4 am: cc1990534a am: 0ade78110e
am: fcf1c416ba

Change-Id: Idd3a2be56689e0f803acf6e78d6ef9ee8c9b3f9f
2017-10-16 01:09:40 +00:00
Dan Willemsen
80ea4ef47c Revert "Always run asan ckati on the build servers" am: 8122bd58ba am: 63f872a239
am: e2cc00a17c

Change-Id: I1f4584db03641faf18cb4fe5875cd13e316bf6e0
2017-10-16 01:07:37 +00:00
Dan Willemsen
fcf1c416ba Always record kati stats into soong log am: 75d2c170b4 am: cc1990534a
am: 0ade78110e

Change-Id: Ia7bb98339bac5c8ce0a6443c3876185b3e977f7d
2017-10-16 01:07:32 +00:00
Dan Willemsen
e2cc00a17c Revert "Always run asan ckati on the build servers" am: 8122bd58ba
am: 63f872a239

Change-Id: I939e3ce6412aee4292b40ac3f1877295d08e93af
2017-10-16 01:05:46 +00:00
Dan Willemsen
0ade78110e Always record kati stats into soong log am: 75d2c170b4
am: cc1990534a

Change-Id: Ide2348aa3c5606d0ed26a052cb1de88e6136a74f
2017-10-16 01:05:40 +00:00
Dan Willemsen
63f872a239 Revert "Always run asan ckati on the build servers"
am: 8122bd58ba

Change-Id: I4cfd5354e3153f23c757095a046f4e57e6ab5aeb
2017-10-16 01:03:41 +00:00
Dan Willemsen
cc1990534a Always record kati stats into soong log
am: 75d2c170b4

Change-Id: I76ffca839f79f38afd10ac26e2ca7d6fb3d261fe
2017-10-16 01:03:37 +00:00
Dan Willemsen
1bb9f30adb Merge "Switch product configs from make to ckati" am: 37cba1471f am: e9d5786b7b am: 43bdf09978
am: aff0e64cf0

Change-Id: Ifb801296d5b3c2c411c2ff043d6fa8763bcd20b7
2017-10-15 04:20:33 +00:00
Jeff Gaston
ea328bd330 Merge "Make some params in soong_zip explicit and/or exported" am: daf774f1f0 am: 821c2c3695 am: bb35f4c2b5
am: d1d56d7ef7

Change-Id: I2168fc020eb0f6dd2d5fcb80961e9f7bad180a92
2017-10-15 04:20:12 +00:00
Elliott Hughes
a92ecc9b36 Merge "Unified sysroot: arch-X/include directories are gone." am: 4d219e801d am: 3808a8c1a6 am: 394944459a
am: d5b4607533

Change-Id: Ibe8e592e76a0ad9203c57aa572e0e31407e4d308
2017-10-15 04:19:53 +00:00
Jiyong Park
5b5854b4ef Squash vendor sources before linkageMutator runs am: 6a43f04777 am: be4d89249a am: 7c7ce296ee
am: dcbdb9e18a

Change-Id: Iafccc06704367f933206d42ec419246675fc1dcf
2017-10-15 04:19:17 +00:00
Dan Willemsen
aff0e64cf0 Merge "Switch product configs from make to ckati" am: 37cba1471f am: e9d5786b7b
am: 43bdf09978

Change-Id: I15cb73423f7f15009570813d9258dc7e90ca2c66
2017-10-15 03:31:44 +00:00
Jeff Gaston
d1d56d7ef7 Merge "Make some params in soong_zip explicit and/or exported" am: daf774f1f0 am: 821c2c3695
am: bb35f4c2b5

Change-Id: Ic6b40a43383f30a41192fbc6a206c5f234c4eeca
2017-10-15 03:31:25 +00:00
Elliott Hughes
d5b4607533 Merge "Unified sysroot: arch-X/include directories are gone." am: 4d219e801d am: 3808a8c1a6
am: 394944459a

Change-Id: I14c2bca2fe1532463b6fe1f3b7ee2bec5b44d025
2017-10-15 03:31:10 +00:00
Jiyong Park
dcbdb9e18a Squash vendor sources before linkageMutator runs am: 6a43f04777 am: be4d89249a
am: 7c7ce296ee

Change-Id: I1c9989c85a2e2a2e58cddbf2b85a0ef8e1de3b6f
2017-10-15 03:30:48 +00:00
Dan Willemsen
43bdf09978 Merge "Switch product configs from make to ckati" am: 37cba1471f
am: e9d5786b7b

Change-Id: Ia798bca03624023e9fe9cc5501f80b9d5bb9cf2b
2017-10-15 03:13:33 +00:00
Jeff Gaston
bb35f4c2b5 Merge "Make some params in soong_zip explicit and/or exported" am: daf774f1f0
am: 821c2c3695

Change-Id: Ie117c866ef419264b38c116ec775d627c13b83fc
2017-10-15 03:13:16 +00:00
Elliott Hughes
394944459a Merge "Unified sysroot: arch-X/include directories are gone." am: 4d219e801d
am: 3808a8c1a6

Change-Id: I3b4f833b700fe262e8e230a40e4ce4da0cb10241
2017-10-15 03:13:00 +00:00
Jiyong Park
7c7ce296ee Squash vendor sources before linkageMutator runs am: 6a43f04777
am: be4d89249a

Change-Id: I6fb3208833bbe99c98d9918a5d0801128513555b
2017-10-15 03:12:42 +00:00
Dan Willemsen
e9d5786b7b Merge "Switch product configs from make to ckati"
am: 37cba1471f

Change-Id: Ia93dec12cfaf067830563757fb7a3b289b7b7c3d
2017-10-15 02:46:56 +00:00
Jeff Gaston
821c2c3695 Merge "Make some params in soong_zip explicit and/or exported"
am: daf774f1f0

Change-Id: Ia892534e3f3f942953605ca8c49a24e1997a10b2
2017-10-15 02:46:33 +00:00
Elliott Hughes
3808a8c1a6 Merge "Unified sysroot: arch-X/include directories are gone."
am: 4d219e801d

Change-Id: I79371e91ebc995f516f4edcdeb7832923877e144
2017-10-15 02:46:15 +00:00
Jiyong Park
be4d89249a Squash vendor sources before linkageMutator runs
am: 6a43f04777

Change-Id: I8656bd768bf82b68821beb28678652bdcf2ebfe6
2017-10-15 02:45:51 +00:00
Dan Willemsen
8122bd58ba Revert "Always run asan ckati on the build servers"
This reverts commit a3e6c520ac. I fixed
a few heavy stack users, and we've got better debugging.

Bug: 36182021
Test: m dist
Change-Id: Iefd8589185720e44c40a2e0ceff5fd1438211b86
2017-10-14 00:43:13 +00:00
Dan Willemsen
75d2c170b4 Always record kati stats into soong log
Pass --dump-stats to ckati, but filter out the '*kati*' lines so that
they only end up in our verbose output. That way we've always got access
to the statistics.

Bug: 36182021
Test: m nothing; cat out/soong.log
Change-Id: Iaf7a814fc67f3e475c913faf69924a7f4e2ae3b3
2017-10-14 00:42:30 +00:00
Dan Willemsen
37cba1471f Merge "Switch product configs from make to ckati" 2017-10-13 23:22:00 +00:00
Jeff Gaston
daf774f1f0 Merge "Make some params in soong_zip explicit and/or exported" 2017-10-13 20:08:52 +00:00
Elliott Hughes
4d219e801d Merge "Unified sysroot: arch-X/include directories are gone." 2017-10-13 17:35:41 +00:00
Jiyong Park
6a43f04777 Squash vendor sources before linkageMutator runs
linkageMutator removes srcs property of the shared variant of a lib in
order to reuse *.o files compiled for the static variant also to the
shared variant.

However, this causes problem when vendor-specific srcs are specified in
target: {vendor: {srcs: ["..."]}}. For example, let's assume

cc_library {
    name: "libfoo",
    srcs: ["foo.c"],
    target: {
        vendor: {
            srcs: ["bar.c"],
        },
    },
}

Then,
static_vendor: inputs = foo.o, bar.o
shared_vendor: inputs = foo.o (from static_vendor), bar.o (from
static_vendor), bar.o

So, bar.o is included twice and this causes multiple symbol definition
error.

In order to handle the problem, vendor mutator is applied before the
linkage mutator and the vendor-specific srcs are squashed in the vendor
mutator.

Bug: 67731122
Test: build
Test: cc_test.go

Change-Id: I2a5390295dddfc41260e9b6f02746908cdf47228
2017-10-13 14:36:12 +09:00
Dan Willemsen
b2e6c2e571 Switch product configs from make to ckati
This speeds up dumping make variables from ~380ms using make to ~220ms
using ckati. It also means that we're consistently using the same parser
for builds (with the same .KATI_READONLY/etc extensions).

envsetup.sh (lunch) / other scripts still use make, changing those to go
through soong_ui will be a future change.

Test: m clean; m nothing
Test: USE_GOMA=true m nothing
Test: m PRODUCT-aosp_x86-sdk
Test: m APP-Calculator
Test: build/soong/build_test.bash -only-config  (on AOSP and internal master)
Change-Id: I6ca554de8de4955fb869001d06d29969b75751cc
2017-10-12 18:11:08 -07:00
Jeff Gaston
c3bdc975a4 Make some params in soong_zip explicit and/or exported
in preparation to split soong_zip into a lib and a bin

Bug: 67478260
Test: m -j
Change-Id: I725da6f0c9b13069e3e5b036e2ac8cbdde9ba6bf
2017-10-12 12:51:52 -07:00
Elliott Hughes
de28deb460 Unified sysroot: arch-X/include directories are gone.
Bug: N/A
Test: builds
Change-Id: I98fc6a1a3c0be1c90b1deb35b54f36ab16a202ac
2017-10-12 09:07:53 -07:00
Colin Cross
9624f6025e Merge "Let tests override environment" am: 432a2d4033 am: 786a19151b am: 6623676b08
am: 3e9df09d7c

Change-Id: I6b7126657130fa57c7cefb14121f90ae9392a3ba
2017-10-11 08:31:29 +00:00
Colin Cross
3e9df09d7c Merge "Let tests override environment" am: 432a2d4033 am: 786a19151b
am: 6623676b08

Change-Id: Ie21e898b1c942dba6fb3571f82a32c1e7d5bfcb3
2017-10-11 08:28:56 +00:00
Colin Cross
6623676b08 Merge "Let tests override environment" am: 432a2d4033
am: 786a19151b

Change-Id: I03066e4e09e4a150b73958d5eb589550097f8cbd
2017-10-11 08:27:00 +00:00
Colin Cross
786a19151b Merge "Let tests override environment"
am: 432a2d4033

Change-Id: I3dc5ba434ad147f94e43c2ccf423d08e50d6de91
2017-10-11 08:23:50 +00:00
Treehugger Robot
432a2d4033 Merge "Let tests override environment" 2017-10-11 08:18:17 +00:00
Colin Cross
6ccbc913e4 Let tests override environment
Pass an environment to TestConfig that will be used for
all Config.Getenv calls.

Test: none
Change-Id: I683cd9c9e0db61c9bfd2adb27fca78f558f225c4
2017-10-10 23:28:43 -07:00