diff options
author | 2016-06-20 22:08:17 +0000 | |
---|---|---|
committer | 2016-06-21 09:59:11 +0000 | |
commit | 1f1f207573c7b9c3e2d3ca1ffb0780a8fd592214 (patch) | |
tree | 4de90d7b6426fece22e4bfd8a5425f226a91f0c9 /src/tools/android/java/com/google/devtools/build/android/BUILD | |
parent | 81a89997be258d6978ead96608c240aee9ec95ea (diff) |
Add action to write android_binary and lib R.classes directly
For android_binary rules, we regenerate all of
the R.java of the libraries to get the final
resource IDs. Compiling all of them together can
be slow with the normal JavaBuilder, so add a
specialized class writer.
Example build with many R.java classes:
- R.java -> R.class via JavaBuilder: over 80s
- ErrorProne takes about 40% of that. So turning off
ErrorProne would be projected to be 48s.
Some of ErrorProne slowness is from static field
checks (e.g., on Flag classes), which may look
up the same Type over and over.
In comparison, if we write our own bytecode with ASM:
- ~16s total
- 4.7s to parse R.txt
- 4.8s to write class files
- 5.8s to copy and compress .jar
TODO: clean up SymbolLoader patching (upstream)
This only creates the action. We will need to
separately wire this up in blaze.
NOTE: This also makes the exising R.txt loading
used by live code multi-threaded, since that is
partly I/O-bound. Something to watch out for
(for flakiness, etc.) once this is submitted.
--
MOS_MIGRATED_REVID=125384467
Diffstat (limited to 'src/tools/android/java/com/google/devtools/build/android/BUILD')
-rw-r--r-- | src/tools/android/java/com/google/devtools/build/android/BUILD | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/src/tools/android/java/com/google/devtools/build/android/BUILD b/src/tools/android/java/com/google/devtools/build/android/BUILD index f4a9a27cee..fe6b36d0b6 100644 --- a/src/tools/android/java/com/google/devtools/build/android/BUILD +++ b/src/tools/android/java/com/google/devtools/build/android/BUILD @@ -26,6 +26,14 @@ java_binary( ) java_binary( + name = "AndroidResourceCompilingAction", + main_class = "com.google.devtools.build.android.AndroidResourceCompilationAction", + runtime_deps = [ + ":android_builder_lib", + ], +) + +java_binary( name = "AndroidResourceProcessingAction", main_class = "com.google.devtools.build.android.AndroidResourceProcessingAction", runtime_deps = [ @@ -59,6 +67,7 @@ java_library( "//src/main/java/com/google/devtools/common/options", "//src/main/protobuf:package_manifest_java_proto", "//src/tools/android/java/com/google/devtools/build/android/proto:serialize_format_proto", + "//src/tools/android/java/com/google/devtools/build/android/resources", "//third_party:android_common", "//third_party:apache_commons_compress", "//third_party:asm", @@ -75,6 +84,7 @@ filegroup( "//src/tools/android/java/com/google/devtools/build/android/idlclass:srcs", "//src/tools/android/java/com/google/devtools/build/android/incrementaldeployment:srcs", "//src/tools/android/java/com/google/devtools/build/android/proto:srcs", + "//src/tools/android/java/com/google/devtools/build/android/resources:srcs", "//src/tools/android/java/com/google/devtools/build/android/ziputils:srcs", ], visibility = ["//src:__pkg__"], |