Commit Graph

1071 Commits

Author SHA1 Message Date
Colin Cross
83964d6f92 Export DefaultsFactory
am: e1d764e369

Change-Id: I5c4b2350707736d018110670c9739b41ba664401
2016-08-23 20:10:57 +00:00
Colin Cross
a120ec1c5e Add PropertyCustomizer
Allow any module factory to insert a PropertyCustomizer on the module,
which will be called before any other mutators.  The PropertyCustomizer
can append or prepend to any properties, allowing module types to extend
other module types by modifying the public, stable interface provided by
the properties.

Change-Id: Idff02be80d939a70df1c6bbccffdd1f04ff975d2
2016-08-22 17:08:06 -07:00
Colin Cross
e1d764e369 Export DefaultsFactory
Add an exported DefaultsFactory so that external module types can extend
cc_defaults.  DefaultsFactory takes a variadic list of additional
property structures.  defaultsFactory remains as the module factory for
cc_defaults.

Change-Id: I03b450ab3a31625e7baca165d393a0007fb3e421
2016-08-22 17:08:06 -07:00
Colin Cross
5bd6252279 Add DeviceConfig and OncePer objects am: 9272ade7a8
am: 8d6e227c10

Change-Id: I57eca6736ab77f0b44b004d6b1dd9e3dbd9ff9d9
2016-08-18 01:06:53 +00:00
Colin Cross
8d6e227c10 Add DeviceConfig and OncePer objects
am: 9272ade7a8

Change-Id: Ida7c815a586b7fa4c27518ddae72fa6b06123be1
2016-08-18 01:04:25 +00:00
Colin Cross
9272ade7a8 Add DeviceConfig and OncePer objects
Add DeviceConfig to store per-device configuration information.  Put a
OncePer object inside Config and DeviceConfig, which computes a value
once per key per object to allow build logic to store arbitrary
per-build or per-device computed values.

Change-Id: I1a38b426f29d223ef5e803e0d4d9604500de2fd2
2016-08-17 16:39:06 -07:00
Colin Cross
c0ab77d8ae Merge "Support required property" am: 389d2bb145
am: 37f1c3dd70

Change-Id: I7b306c7836cf1f15e07c62d3c6e74db0a7dae68e
2016-08-17 17:25:52 +00:00
Colin Cross
37f1c3dd70 Merge "Support required property"
am: 389d2bb145

Change-Id: I1f63a850a3c497dec170e85606b0f1e1545b19fe
2016-08-17 17:23:22 +00:00
Colin Cross
389d2bb145 Merge "Support required property" 2016-08-17 17:06:22 +00:00
Evgenii Stepanov
27faca7e4e Add ubsan_standalone library name to the toolchain. am: af36db1178
am: d820bdeda7

Change-Id: I001e87ffd49b6e7e7c476b3280a5c6ea7663fc60
2016-08-16 21:10:19 +00:00
Evgenii Stepanov
d820bdeda7 Add ubsan_standalone library name to the toolchain.
am: af36db1178

Change-Id: I110f5e295ff2c66838ea6b3e998ea04e84e08a62
2016-08-16 21:07:50 +00:00
Evgenii Stepanov
af36db1178 Add ubsan_standalone library name to the toolchain.
and export the library name to make. Refactor the code a bit to avoid repeating the library name
multiple times.

Bug: 22033465
Test: Ran external/clang/build.py for aosp-llvm

Change-Id: I25eb3858eb92e1dd493b09524d559802551b2547
2016-08-15 17:05:04 -07:00
Chris Wolfe
998306e525 Support required property
This adds LOCAL_REQUIRED_MODULES to the generated Android.mk for any
dependencies in the "required" list of a module.

Change-Id: Ibcfe89dce7d247eb441af94e37388a59e71a75a9
2016-08-15 17:10:47 -04:00
Colin Cross
f77af52934 Merge "Fix coverage sanitizer builds" am: 9883461664
am: d6bf916a03

Change-Id: I73bca60ce3ab4ce1e175e5aa626f98b25a51d8d5
2016-08-15 18:29:08 +00:00
Colin Cross
d6bf916a03 Merge "Fix coverage sanitizer builds"
am: 9883461664

Change-Id: I2b5ab662ff95c5e18827750fde9bd8f72d24777e
2016-08-15 18:24:40 +00:00
Colin Cross
9883461664 Merge "Fix coverage sanitizer builds" 2016-08-15 18:12:23 +00:00
Colin Cross
34439a5140 Fix cc_benchmark installs am: 33586f3b10
am: 7d7245f3fe

Change-Id: I2c6633b05a271b35f647d2e905213e4d90686ffe
2016-08-12 06:55:48 +00:00
Colin Cross
7d7245f3fe Fix cc_benchmark installs
am: 33586f3b10

Change-Id: I51cad975fb5d556e01645fe05ea20d3ca7c5e309
2016-08-12 06:52:50 +00:00
Colin Cross
33586f3b10 Fix cc_benchmark installs
cc_benchmark was not propagating the call to baseInstaller.install,
which resulted in no install location being set and the benchmarks
installed in out/.

Change-Id: I4399cd479d9cd7ec1b7332dd0d20a9ab3820b04e
2016-08-11 22:26:17 -07:00
Colin Cross
f244b4c897 Make defaults TopDownMutator parallel am: 76f2f97c51
am: bb3d66c1fa

Change-Id: I4933e3348cbdfb7bd720139ee972d0c269d4d509
2016-08-12 02:52:04 +00:00
Colin Cross
bb3d66c1fa Make defaults TopDownMutator parallel
am: 76f2f97c51

Change-Id: I40d5eb0cf8721ef89c92ebbd2bf9f75fce30381a
2016-08-12 02:48:37 +00:00
Colin Cross
76f2f97c51 Make defaults TopDownMutator parallel
Append .Parallel() to the defaults RegisterTopDownMutator call to tell
Blueprint it can run it in parallel.  Saves ~500ms in soong_build.

Change-Id: I43ddd9d6995674ccc06fed6928514f15a15712c1
2016-08-11 17:15:07 -07:00
Colin Cross
91169fe8ea Fix coverage sanitizer builds
Disable coverage for static binaries where address sanitizer is
disabled, and disable coverage in the mutator when disabling address
sanitizer.

Bug: 29188876
Change-Id: Ia1a21878c3f34cd295a6dec49608c412eb09e7b1
2016-08-11 16:54:01 -07:00
Colin Cross
d81339cf95 Merge "Make BottomUpMutators parallel" am: aa99f84e57
am: e0b666c229

Change-Id: Ia75c3a485ba26a4dd83753e488460b754eb877f6
2016-08-11 21:48:45 +00:00
Colin Cross
e0b666c229 Merge "Make BottomUpMutators parallel"
am: aa99f84e57

Change-Id: I49cd56f16258f5405515275309bcb0329a32e639
2016-08-11 21:45:17 +00:00
Treehugger Robot
aa99f84e57 Merge "Make BottomUpMutators parallel" 2016-08-11 21:35:24 +00:00
Colin Cross
e8a67a7c58 Make BottomUpMutators parallel
Append .Parallel() to all of the RegisterBottomUpMutator calls to tell
Blueprint it can run them in parallel.

Test: identical build.ninja, passes race detector
Change-Id: I969a0689522d4cba7c8ff51e2aa00fe2fd338a89
2016-08-11 13:20:16 -07:00
Dan Willemsen
e037b7a60b Merge changes from topic 'fast_dist' am: 8bdd631a75
am: da8a307625

Change-Id: I1225b0d96ef241537ba3f50d441eb7f2b39459c0
2016-08-11 19:36:45 +00:00
Dan Willemsen
da8a307625 Merge changes from topic 'fast_dist'
am: 8bdd631a75

Change-Id: I4a3e8b06e155d8c7f830b4db1790461943278d8c
2016-08-11 19:33:14 +00:00
Treehugger Robot
8bdd631a75 Merge changes from topic 'fast_dist'
* changes:
  Rename soong_jar to soong_zip
  soong_jar: Parallel compression
  Add zip2zip tool to copy zip entries from one file to another
  Add archive/zip from go1.7rc5 tag
2016-08-11 19:28:14 +00:00
Dan Albert
1f243d67a3 Add support for NDK r13. am: d015c4ac1b
am: 585738ad21

Change-Id: I7f7d2dc7a0fc1fcb4b9b804bc86126427c481886
2016-08-11 16:56:10 +00:00
Dan Albert
585738ad21 Add support for NDK r13.
am: d015c4ac1b

Change-Id: I1d1c599bbe918537fb4afd4ca568937ee7859923
2016-08-11 16:52:59 +00:00
Dan Willemsen
47ec28f3b1 Rename soong_jar to soong_zip
This is a general purpose tool that happens to contain some jar specific
features.

Change-Id: I05f4654d4517c245ad7a3c15492e0d2368bbf64f
2016-08-11 01:30:16 -07:00
Dan Willemsen
017d8937fa soong_jar: Parallel compression
This compresses multiple files in parallel, and will split up larger
files (5MB+) into smaller chunks (1MB) to compress in parallel.

There is a small size overhead to recombine the chunks, but it's only a
few bytes per chunk, so for a 1MB chunk, it's minimal.

Rough numbers, with everything in the page cache, this can compress
~4GB (1000 files) down to 1GB in 6.5 seconds, instead of 120 seconds with
the non-parallel soong_jar and 150 seconds with zip.

Go's DEFLATE algorithm is still a bit worse than zip's -- about 3.5%
larger file sizes, but for most of our "dist" targets that is fine.

Change-Id: Ie4886c7d0f954ace46e599156e35fea7e74d6dd7
2016-08-11 01:30:16 -07:00
Dan Albert
d015c4ac1b Add support for NDK r13.
Test: make checkbuild
Bug: None
Change-Id: I6e7c92d6731be4ae47c7dc5e9444af08d338334b
2016-08-10 23:14:13 -07:00
Dan Willemsen
3bf1a08505 Add zip2zip tool to copy zip entries from one file to another
This doesn't do any decompression / recompression, but just copies over
the already compressed contents. So it's similar to zip -U, but allows
rewriting of the paths.

The first expected usecase is to replace img_from_target_files during
the build, since it does the equivalent of this:

zip2zip -i <target-files.zip> -o <img.zip> OTA/android-info.txt:android-info.txt IMAGES/*:.

Except it decompresses and recompresses the images, which takes over a
minute instead of a few seconds.

Change-Id: I88d0df188635088783223873f78e193272dbdf1c
2016-08-10 16:57:03 -07:00
Dan Willemsen
25a4e07df8 Add archive/zip from go1.7rc5 tag
In preparation to patch in some custom functionality (parallel
compression and zero-decompress zip to zip copying)

Change-Id: I96a36efc09c715f6b55290af40ebfdde9ae72e33
2016-08-10 16:10:16 -07:00
Colin Cross
85d275aea5 Convert defaults to a top down mutator am: 1317701114
am: adbf48c3df

Change-Id: Ib99c9e34876abc8ecd8ba01ccf2fe9eaa2b5a7c8
2016-08-09 23:25:34 +00:00
Colin Cross
adbf48c3df Convert defaults to a top down mutator
am: 1317701114

Change-Id: I3736f583432d56a7dd1889ba948b9253158c9543
2016-08-09 23:23:01 +00:00
Colin Cross
3acee1544f Update build.ninja.in for blueprint fs.go changes am: 6debdbad42
am: 476861290e

Change-Id: I861f4d9ab81271fcd0802e062b0b9756b854860e
2016-08-09 22:09:32 +00:00
Colin Cross
476861290e Update build.ninja.in for blueprint fs.go changes
am: 6debdbad42

Change-Id: I2fd3171f980ff02ff2b7665f80fdbed3535f489e
2016-08-09 22:07:06 +00:00
Colin Cross
1317701114 Convert defaults to a top down mutator
BottomUpMutators are going to lose their ctx.Visit* functions in order
to allow parallelizing them, move defaults to a TopDownMutator using
WalkDeps to only visit defaults modules.

Test: no changes to out/soong/build.ninja
Change-Id: I54ba65a7e2ae9503f4d217f63aa9178a7c5341f0
2016-08-09 13:54:03 -07:00
Colin Cross
6debdbad42 Update build.ninja.in for blueprint fs.go changes
Change-Id: I05bca49a563b13731ccb89d6f073edd59fee7049
2016-08-09 13:40:08 -07:00
Colin Cross
4fa0a9debc Merge "Use nil pointers for arch property structs" am: 142d4fd01d
am: 3099506f27

Change-Id: I6498fb154ead628568b7b01965de5183747c30b6
2016-08-09 00:22:15 +00:00
Colin Cross
3099506f27 Merge "Use nil pointers for arch property structs"
am: 142d4fd01d

Change-Id: I006c7325e0b16aeeb947788545184fc96e018cf5
2016-08-09 00:19:18 +00:00
Treehugger Robot
142d4fd01d Merge "Use nil pointers for arch property structs" 2016-08-09 00:08:23 +00:00
Dan Albert
3342e46e9c More fixes for decorator refactor. am: cdd4c24ff7
am: 0f2f0fa282

Change-Id: Ia13b4d4367a7866ff688d291ea6aa07e27f6d5a2
2016-08-08 23:16:00 +00:00
Dan Albert
0f2f0fa282 More fixes for decorator refactor.
am: cdd4c24ff7

Change-Id: I169c2e8271d5e446f1a8a7fd358672c60c6b93c0
2016-08-08 23:13:01 +00:00
Colin Cross
62496a0d2e Use nil pointers for arch property structs
Blueprint can now handle creating a zero-valued struct when it
encounters a nil pointer to a struct, reducing the amount of allocations
and copying for arch structures that are not used.

Change-Id: If36da5603dbe6469fe6406c821f21a122ea6ea49
2016-08-08 15:56:19 -07:00
Dan Albert
cdd4c24ff7 More fixes for decorator refactor.
Didn't catch these the first time because I hadn't done a clean build.

Test: rm -rf out && make ndk
Change-Id: Ia66ad614d2163272889ee6937a6449c44db16ac0
2016-08-08 14:45:24 -07:00