NPE in Enqueuer releated to record DexCallSite [381812767]

Fixed

Bug

Status Update

No update yet.

Description

sg...@google.com

created issue #1

Dec 2, 2024 09:45AM

No description yet.

Comments

ap...@google.com <ap...@google.com> #2Dec 2, 2024 12:46PM

Project: r8
Branch: main

commit 39f08f72219210c78d964f356552efd6c8a3275a
Author: Morten Krogh-Jespersen <mkroghj@google.com>
Date: Thu Jan 19 14:12:31 2023

[Metadata] Update kotlin-metadata-jvm to version 0.6.0

Bug:

b/265108171

Change-Id: I2441b7c328acda8efa13c4e0a7c334cf275f0b44

M build.gradle
M src/main/java/com/android/tools/r8/kotlin/KotlinClassInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinClassLevelInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinClassMetadataReader.java
M src/main/java/com/android/tools/r8/kotlin/KotlinFileFacadeInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinMetadataRewriter.java
M src/main/java/com/android/tools/r8/kotlin/KotlinMetadataUtils.java
M src/main/java/com/android/tools/r8/kotlin/KotlinMultiFileClassFacadeInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinMultiFileClassPartInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinSyntheticClassInfo.java
M src/test/java/com/android/tools/r8/kotlin/coroutines/KotlinxCoroutinesTestRunner.java
M src/test/java/com/android/tools/r8/kotlin/metadata/KotlinMetadataTestBase.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteBoxedTypesTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteInlineClassTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteInlinePropertyTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteKeepTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteLocalDelegatedPropertyTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewritePassThroughTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteRawTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteUnitPrimitiveTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteValueClassTest.java

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

ch...@google.com <ch...@google.com> Dec 3, 2024 03:51PM

Reassigned to ch...@google.com.

ap...@google.com <ap...@google.com> #3Dec 4, 2024 09:46AM

Marked as fixed.

Project: r8
Branch: main

commit dde8cf4cd2e452225400d77819767cd48406cd41
Author: Morten Krogh-Jespersen <mkroghj@google.com>
Date: Mon Jan 23 17:32:03 2023

[Metadata] Update service import in sanity check test

Bug:

b/265108171

Change-Id: I25d5a272b8a3cab18e638e85553bbf5b98997b60

M src/test/java/com/android/tools/r8/SanityCheck.java

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

ap...@google.com <ap...@google.com> #4Dec 4, 2024 11:36AM

The public work of this has now been completed. I created b/266682027 for tracking the internal work on upgrading the library.

ap...@google.com <ap...@google.com> #5Dec 4, 2024 11:36AM

Project: r8
Branch: 8.0

commit 177f2a32494e4e9baedd895eb920bc41825ef77b
Author: Morten Krogh-Jespersen <mkroghj@google.com>
Date: Wed Jan 25 13:22:14 2023

Version 8.0.27

Bug:

b/265108171

Bug:

b/266177488

Bug:

b/266396725

Change-Id: Iebc28bdd06d6330116a8b12336f25be201603745

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

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

ap...@google.com <ap...@google.com> #6Dec 4, 2024 11:36AM

Project: r8
Branch: 8.0

commit 1c9ab7b7cff3bdc3de67ce8b6cd032907f72c8ef
Author: Morten Krogh-Jespersen <mkroghj@google.com>
Date: Wed Jan 25 13:21:40 2023

[Metadata] Update service import in sanity check test

Bug:

b/265108171

Change-Id: I25d5a272b8a3cab18e638e85553bbf5b98997b60

M src/test/java/com/android/tools/r8/SanityCheck.java

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

ap...@google.com <ap...@google.com> #7Dec 4, 2024 11:37AM

Project: r8
Branch: 8.0

commit 1db3d77589ae2bc16ba2501f9848d40afb780139
Author: Morten Krogh-Jespersen <mkroghj@google.com>
Date: Wed Jan 25 13:21:27 2023

Reland "[Metadata] Update kotlin-metadata-jvm to version 0.6.0"

This reverts commit 87b372dde22f9052f9fb86770921bdf6cb008ba6.

Change-Id: I7b361c05a686fbfffd399f5a7ee10f65c8513b2f
Bug:

b/265108171

M build.gradle
M src/main/java/com/android/tools/r8/kotlin/KotlinClassInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinClassLevelInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinClassMetadataReader.java
M src/main/java/com/android/tools/r8/kotlin/KotlinFileFacadeInfo.java
A src/main/java/com/android/tools/r8/kotlin/KotlinMetadataAnnotationWrapper.java
M src/main/java/com/android/tools/r8/kotlin/KotlinMetadataRewriter.java
M src/main/java/com/android/tools/r8/kotlin/KotlinMetadataUtils.java
M src/main/java/com/android/tools/r8/kotlin/KotlinMultiFileClassFacadeInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinMultiFileClassPartInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinSyntheticClassInfo.java
M src/test/java/com/android/tools/r8/kotlin/coroutines/KotlinxCoroutinesTestRunner.java
M src/test/java/com/android/tools/r8/kotlin/metadata/KotlinMetadataTestBase.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteBoxedTypesTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteInlineClassTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteInlinePropertyTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteKeepTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteLocalDelegatedPropertyTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewritePassThroughTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteRawTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteUnitPrimitiveTest.java
M src/test/java/com/android/tools/r8/kotlin/metadata/MetadataRewriteValueClassTest.java

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

ap...@google.com <ap...@google.com> #8Dec 4, 2024 11:37AM

Project: r8
Branch: main

commit 4ff176f892492b8bbb1a647f83cc0a9e679b40d9
Author: Morten Krogh-Jespersen <mkroghj@google.com>
Date: Thu Jan 26 09:58:05 2023

[Metadata] Move away from deprecated metadata visitors

Bug:

b/265108171

Change-Id: I3cd8f4b0a78ef6bfe9d9c38e2629b89315d0c9d1

D src/main/java/com/android/tools/r8/kotlin/KmVisitorProviders.java
M src/main/java/com/android/tools/r8/kotlin/KotlinAnnotationInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinClassInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinConstructorInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinContractInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinDeclarationContainerInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinEffectExpressionInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinEffectInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinFlexibleTypeUpperBoundInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinFunctionInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinJvmMethodSignatureInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinLambdaInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinLocalDelegatedPropertyInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinMetadataUtils.java
M src/main/java/com/android/tools/r8/kotlin/KotlinPackageInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinPropertyInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinSyntheticClassInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinTypeAliasInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinTypeInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinTypeParameterInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinTypeProjectionInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinValueParameterInfo.java
M src/main/java/com/android/tools/r8/kotlin/KotlinVersionRequirementInfo.java

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

ap...@google.com <ap...@google.com> #9Dec 4, 2024 11:37AM

Project: r8
Branch: 8.6
Author: Christoffer Adamsen <christofferqa@google.com>
Link: https://r8-review.googlesource.com/99083

Version 8.6.37

Expand for full commit details

Version 8.6.37 
 
Bug: b/381812767 
Change-Id: Ibf1ad407fdd2c844db37b12e12e0c5d1b53fe6b1

Files:

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

Hash: 2790fc1631674ab8300c9c61826822d695d9063f
Date: Wed Dec 04 12:04:37 2024

ap...@google.com <ap...@google.com> #10Dec 4, 2024 11:41AM

Project: r8
Branch: 8.8
Author: Christoffer Adamsen <christofferqa@google.com>
Link: https://r8-review.googlesource.com/99080

Fix cf-to-cf handling of record call sites

Expand for full commit details

Fix cf-to-cf handling of record call sites 
 
Fixes: b/381812767 
Change-Id: Ibdbf5ccd40d8322eb4953343efb923434e396e22

Files:

M src/main/java/com/android/tools/r8/optimize/argumentpropagation/ArgumentPropagatorMethodReprocessingEnqueuer.java

Hash: 770d64ea75f169e72cf2612c8ca06709725664d1
Date: Wed Dec 04 12:03:44 2024

ap...@google.com <ap...@google.com> #11Dec 4, 2024 11:41AM

Project: r8
Branch: 8.7
Author: Christoffer Adamsen <christofferqa@google.com>
Link: https://r8-review.googlesource.com/99082

Fix cf-to-cf handling of record call sites

Expand for full commit details

Fix cf-to-cf handling of record call sites 
 
Fixes: b/381812767 
Change-Id: Ibdbf5ccd40d8322eb4953343efb923434e396e22

Files:

M src/main/java/com/android/tools/r8/optimize/argumentpropagation/ArgumentPropagatorMethodReprocessingEnqueuer.java

Hash: 0071dc419efcf442b4680778e30f030221ebc36e
Date: Wed Dec 04 12:04:07 2024

an...@google.com <an...@google.com> #12Feb 13, 2025 06:23PM

Thank you for your patience while our engineering team worked to resolve this issue. A fix for this issue is now available in:

Android Studio Ladybug Feature Drop | 2024.2.2 Patch 1
Android Gradle Plugin 8.8.1

We encourage you to try the latest update.

If you notice further issues or have questions, please file a new bug report.

Thank you for taking the time to submit feedback — we really appreciate it!

Issue 381812767

Description

Issue summary

Comments

ap...@google.com <ap...@google.com> #2Dec 2, 2024 12:46PM

ch...@google.com <ch...@google.com> Dec 3, 2024 03:51PM

ap...@google.com <ap...@google.com> #3Dec 4, 2024 09:46AM

ap...@google.com <ap...@google.com> #4Dec 4, 2024 11:36AM

ap...@google.com <ap...@google.com> #5Dec 4, 2024 11:36AM

ap...@google.com <ap...@google.com> #6Dec 4, 2024 11:36AM

ap...@google.com <ap...@google.com> #7Dec 4, 2024 11:37AM

ap...@google.com <ap...@google.com> #8Dec 4, 2024 11:37AM

ap...@google.com <ap...@google.com> #9Dec 4, 2024 11:37AM

ap...@google.com <ap...@google.com> #10Dec 4, 2024 11:41AM

ap...@google.com <ap...@google.com> #11Dec 4, 2024 11:41AM

an...@google.com <an...@google.com> #12Feb 13, 2025 06:23PM

Add comment

Issue metadata