Minifying Renderscript code: huge performance drop when upgrading AGP from 7.3.1 to 7.4.0 [267990059]

Verified

Bug

Status Update

No update yet.

Description

bv...@gmail.com

created issue #1

Feb 6, 2023 09:26AM

We're using Renderscript in one of our projects and noticed that our build times for a minified release build went up by a factor of 3 after upgrading the Android Gradle Plugin from 7.3.1 to 7.4.0: from 17 minutes to 53 minutes.

For each Renderscript file, two files are generated: a ScriptC_$name file which contains bridging functions, and a $nameBitCode file which contains a few huge byte arrays that are compiled into GPU code on the user's phone. R8 takes a very long time to process this last file.

I made a sample project here: https://github.com/bvschaik/r8-renderscript-minify-issue
This project contains a copy of the BitCode class that's generated by Renderscript here.

Using AGP 7.4.0 (uses R8 4.0.48), running assembleRelease on this project takes almost 4 minutes, with 3m40s taken up by the minifyReleaseWithR8 task.

Downgrading AGP to 7.3.1 (uses R8 3.3.83), the minifyReleaseWithR8 "only" takes 1m10s, which is still quite long for processing some static byte arrays.

For now we're forcing R8 to the last available 3.3 version to get back to acceptable build times.

What could cause this increase in build times?
Why is R8 spending so much time on relatively simple code?
What can we do to tell R8 "don't change anything in this file"? I tried various -keep rules but it didn't make a difference.

Comments

ch...@google.com <ch...@google.com> Feb 6, 2023 09:35AM

Reassigned to ch...@google.com.

ch...@google.com <ch...@google.com> #2Feb 7, 2023 07:57AM

I also have the same issue so it seems to be a real problem.

ap...@google.com <ap...@google.com> #3Feb 8, 2023 11:31AM

Dup here:

http://code.google.com/p/android/issues/detail?id=27959&can=1&q=fixed%20position&colspec=ID%20Type%20Status%20Owner%20Summary%20Stars

Though it's a bit more detailed with the minimum necessary to reproduce.

ap...@google.com <ap...@google.com> #4Feb 8, 2023 02:53PM

why is this marked has Obsolete? this is still a bug that is happening and its difficult to fix.
for all who are looking for a solution use this:

http://stackoverflow.com/questions/16200972/android-css-positionfixed-after-a-device-rotate

ap...@google.com <ap...@google.com> #5Feb 8, 2023 03:16PM

Project: r8
Branch: main

commit 70bbb88b80b31de9a5ccbe30d27a962b0fb0c45a
Author: Christoffer Quist Adamsen <christofferqa@google.com>
Date: Wed Feb 08 16:13:24 2023

Fix repeated analysis of values in dead code removal

Bug:

b/267990059

Change-Id: I9deebf700e936ad81a8fb25d067acc04637d980d

M src/main/java/com/android/tools/r8/ir/code/ArrayPut.java
M src/main/java/com/android/tools/r8/ir/code/ValueIsDeadAnalysis.java
M src/main/java/com/android/tools/r8/ir/optimize/DeadCodeRemover.java
M src/main/java/com/android/tools/r8/utils/MapUtils.java

https://r8-review.googlesource.com/75682

ap...@google.com <ap...@google.com> #6Feb 9, 2023 10:06AM

Project: r8
Branch: main

commit 54ed0312d2e5ade4b8092a86989d0e6827e646dd
Author: Christoffer Quist Adamsen <christofferqa@google.com>
Date: Thu Feb 09 08:49:58 2023

Refinements to value-is-dead analysis

Bug:

b/267990059

Fixes:

b/268335684

Change-Id: I8007b13e2a4e0c32dffc181a827867ff4ab0373e

M src/main/java/com/android/tools/r8/ir/code/ValueIsDeadAnalysis.java
M src/main/java/com/android/tools/r8/ir/optimize/DeadCodeRemover.java

https://r8-review.googlesource.com/75761

ap...@google.com <ap...@google.com> #7Feb 9, 2023 10:27AM

Project: r8
Branch: 4.0

commit 2fc27f8cef33c22525c585e5c65bf3d67a18755a
Author: Christoffer Quist Adamsen <christofferqa@google.com>
Date: Thu Feb 09 11:19:09 2023

Version 4.0.53

Bug:

b/267990059

Bug:

b/267463817

Change-Id: I62f98b9723358310ade7f2bd5f8e589fa91c8ab6

M src/main/java/com/android/tools/r8/Version.java

https://r8-review.googlesource.com/75769

ap...@google.com <ap...@google.com> #8Feb 9, 2023 10:27AM

Project: r8
Branch: 4.0

commit b86fdf2b2427605e6a238a0232932fa1268245f8
Author: Christoffer Quist Adamsen <christofferqa@google.com>
Date: Thu Feb 09 11:16:04 2023

Refinements to value-is-dead analysis

Bug:

b/267990059

Fixes:

b/268335684

Change-Id: I8007b13e2a4e0c32dffc181a827867ff4ab0373e

M src/main/java/com/android/tools/r8/ir/code/ValueIsDeadAnalysis.java
M src/main/java/com/android/tools/r8/ir/optimize/DeadCodeRemover.java

https://r8-review.googlesource.com/75767

ap...@google.com <ap...@google.com> #9Feb 9, 2023 10:27AM

Project: r8
Branch: 4.0

commit 472c7fab11e7da906d07eb04d8600f6aea8d3c1e
Author: Christoffer Quist Adamsen <christofferqa@google.com>
Date: Thu Feb 09 11:15:38 2023

Fix repeated analysis of values in dead code removal

Bug:

b/267990059

https://r8-review.googlesource.com/75766

ap...@google.com <ap...@google.com> #10Feb 9, 2023 10:27AM

Project: r8
Branch: 4.0

commit b14e5bc4c5f41fc593cc6176de220ce43d9c899f
Author: Christoffer Quist Adamsen <christofferqa@google.com>
Date: Thu Feb 09 11:15:14 2023

Rewrite value-is-dead analysis to a worklist algorithm

Bug:

b/267990059

Change-Id: I699f43eba8774083a4b5f2aa2edf7c5d80df4c8e

M src/main/java/com/android/tools/r8/ir/code/ValueIsDeadAnalysis.java

https://r8-review.googlesource.com/75765

ap...@google.com <ap...@google.com> #11Feb 9, 2023 10:27AM

Project: r8
Branch: 4.0

commit 4621177fe24819fe5459b065a9d6d138cb79b5b2
Author: Christoffer Quist Adamsen <christofferqa@google.com>
Date: Thu Feb 09 11:14:33 2023

Refactor value-is-dead analysis to separate analysis class

Bug:

b/267990059

Change-Id: I8b881203ebe21395f48c8eeb6434faa03f7f9d46

M src/main/java/com/android/tools/r8/ir/code/BasicBlock.java
M src/main/java/com/android/tools/r8/ir/code/Value.java
A src/main/java/com/android/tools/r8/ir/code/ValueIsDeadAnalysis.java
M src/main/java/com/android/tools/r8/ir/optimize/DeadCodeRemover.java

https://r8-review.googlesource.com/75764

ap...@google.com <ap...@google.com> #12Feb 9, 2023 10:30AM

Project: r8
Branch: 8.0

commit 376af144cf327f0f2b0a2de0a38939a59216a08d
Author: Christoffer Quist Adamsen <christofferqa@google.com>
Date: Thu Feb 09 11:25:18 2023

Version 8.0.29

Bug:

b/267990059

Bug:

b/267463817

Change-Id: I994d406bae7514482ffdf2ef10f092b5a96cc328

M src/main/java/com/android/tools/r8/Version.java

https://r8-review.googlesource.com/75776

ap...@google.com <ap...@google.com> #13Feb 9, 2023 10:30AM

Project: r8
Branch: 8.0

commit 77704f63db4161833b7072c42cbf9869a2f20e80
Author: Christoffer Quist Adamsen <christofferqa@google.com>
Date: Thu Feb 09 11:23:20 2023

Refinements to value-is-dead analysis

Bug:

b/267990059

Fixes:

b/268335684

Change-Id: I8007b13e2a4e0c32dffc181a827867ff4ab0373e

M src/main/java/com/android/tools/r8/ir/code/ValueIsDeadAnalysis.java
M src/main/java/com/android/tools/r8/ir/optimize/DeadCodeRemover.java

https://r8-review.googlesource.com/75773

ap...@google.com <ap...@google.com> #14Feb 9, 2023 10:30AM

Project: r8
Branch: 8.0

commit 33930d7b7ff37a746a304bb0230b34123b1e0e6f
Author: Christoffer Quist Adamsen <christofferqa@google.com>
Date: Thu Feb 09 11:22:50 2023

Fix repeated analysis of values in dead code removal

Bug:

b/267990059

https://r8-review.googlesource.com/75772

ap...@google.com <ap...@google.com> #15Feb 9, 2023 10:30AM

Project: r8
Branch: 8.0

commit 923cf259db1b2f35578a486960afcd28369c520f
Author: Christoffer Quist Adamsen <christofferqa@google.com>
Date: Thu Feb 09 11:22:16 2023

Rewrite value-is-dead analysis to a worklist algorithm

Bug:

b/267990059

Change-Id: I699f43eba8774083a4b5f2aa2edf7c5d80df4c8e

M src/main/java/com/android/tools/r8/ir/code/ValueIsDeadAnalysis.java

https://r8-review.googlesource.com/75771

ap...@google.com <ap...@google.com> #16Feb 9, 2023 10:30AM

Project: r8
Branch: 8.0

commit 736dd75501862e23764d265887a99ef8d8af0c9d
Author: Christoffer Quist Adamsen <christofferqa@google.com>
Date: Thu Feb 09 11:21:02 2023

Refactor value-is-dead analysis to separate analysis class

Bug:

b/267990059

https://r8-review.googlesource.com/75770

ch...@google.com <ch...@google.com> #17Feb 9, 2023 10:34AM

Verified by ch...@google.com.

This should now be fixed in R8 versions 4.0.53 and 8.0.29 - and next week in 8.1.19-dev.

On my machine the build time for compiling the reproduction with R8 is down from ~190s to ~3s.

bv...@gmail.com <bv...@gmail.com> #18Feb 9, 2023 12:15PM

I just tested 4.0.53: on the sample project it also runs in ~3s in my machine. The larger project now finishes within a few minutes, and that previously took almost an hour, wow. Thanks!

Any idea when this fix will become available in the AGP? Next release of Android Studio, I assume?

ch...@google.com <ch...@google.com> #19Feb 9, 2023 02:24PM

Thanks for the confirmation. This will hopefully make it into Flamingo (AGP 8.0).

Issue 267990059

Description

Issue summary

Comments

ch...@google.com <ch...@google.com> Feb 6, 2023 09:35AM

ch...@google.com <ch...@google.com> #2Feb 7, 2023 07:57AM

ap...@google.com <ap...@google.com> #3Feb 8, 2023 11:31AM

ap...@google.com <ap...@google.com> #4Feb 8, 2023 02:53PM

ap...@google.com <ap...@google.com> #5Feb 8, 2023 03:16PM

ap...@google.com <ap...@google.com> #6Feb 9, 2023 10:06AM

ap...@google.com <ap...@google.com> #7Feb 9, 2023 10:27AM

ap...@google.com <ap...@google.com> #8Feb 9, 2023 10:27AM

ap...@google.com <ap...@google.com> #9Feb 9, 2023 10:27AM

ap...@google.com <ap...@google.com> #10Feb 9, 2023 10:27AM

ap...@google.com <ap...@google.com> #11Feb 9, 2023 10:27AM

ap...@google.com <ap...@google.com> #12Feb 9, 2023 10:30AM

ap...@google.com <ap...@google.com> #13Feb 9, 2023 10:30AM

ap...@google.com <ap...@google.com> #14Feb 9, 2023 10:30AM

ap...@google.com <ap...@google.com> #15Feb 9, 2023 10:30AM

ap...@google.com <ap...@google.com> #16Feb 9, 2023 10:30AM

ch...@google.com <ch...@google.com> #17Feb 9, 2023 10:34AM

bv...@gmail.com <bv...@gmail.com> #18Feb 9, 2023 12:15PM

ch...@google.com <ch...@google.com> #19Feb 9, 2023 02:24PM

Add comment

Issue metadata