commit a88a0dbca042fe6e9587ec66fc7e934b530cb2c5
author Clément Béra <clementbera@google.com> Wed Dec 13 10:47:14 2023 +0100
committer Clément Béra <clementbera@google.com> Wed Dec 13 10:11:37 2023 +0000
tree 5e44f119254ad3d253f004037cd2c60869bb0a54
parent e42b7e1fbe6755cc77257fff060afe98ffa8b045

Fix abstract method

Change-Id: Iaac8c33c543dff310d21df89e00a48e7c649593b

src/main/java/com/android/tools/r8/ir/optimize/numberunboxer/NumberUnboxerImpl.java

diff --git a/src/main/java/com/android/tools/r8/ir/optimize/numberunboxer/NumberUnboxerImpl.java b/src/main/java/com/android/tools/r8/ir/optimize/numberunboxer/NumberUnboxerImpl.java
index 29fd7d5..2b30686 100644
--- a/src/main/java/com/android/tools/r8/ir/optimize/numberunboxer/NumberUnboxerImpl.java
+++ b/src/main/java/com/android/tools/r8/ir/optimize/numberunboxer/NumberUnboxerImpl.java
@@ -110,7 +110,8 @@
     Map<DexMethod, DexMethod> vMethodRepresentative = new IdentityHashMap<>();
     for (List<ProgramMethod> vMethods : componentVirtualMethods.values()) {
       if (vMethods.size() > 1) {
-        if (Iterables.all(vMethods, this::shouldConsiderForUnboxing)) {
+        if (Iterables.all(vMethods, this::shouldConsiderForUnboxing)
+            && Iterables.any(vMethods, m -> !m.getDefinition().isAbstract())) {
           vMethods.sort(Comparator.comparing(DexClassAndMember::getReference));
           ProgramMethod representative = vMethods.get(0);
           for (int i = 1; i < vMethods.size(); i++) {
@@ -122,7 +123,7 @@
       } else {
         assert vMethods.size() == 1;
         ProgramMethod candidate = vMethods.get(0);
-        if (shouldConsiderForUnboxing(candidate)) {
+        if (shouldConsiderForUnboxing(candidate) && !candidate.getDefinition().isAbstract()) {
           candidateBoxingStatus.put(candidate.getReference(), UNPROCESSED_CANDIDATE);
         }
       }