IssueTracker

Thank you for the report and reproduction. I can reproduce on a Android TV emulator.

Looks like dead code removal in java.lang.Object androidx.collection.LruCache.remove(java.lang.Object) is causing this.

The method java.lang.Object androidx.collection.LruCache.remove(java.lang.Object) is processed twice in first round of IR, second time with more precise type for the argument as java.lang.Object androidx.collection.LruCache.remove(java.lang.String).

That second processing involves quite a bit of inlining. However before dead code removal the return is still split between null and not null return value:

block 33, pred-counts: 1, succ-count: 2, filled: true, sealed: false
predecessors: 32 
successors: 37 34  (no try/catch successors)
no phis
 #187:remove;0:remove: -1: If                   v54 EQZ block 37 (fallthrough 34)

block 34, pred-counts: 1, succ-count: 1, filled: true, sealed: false
predecessors: 33 
successors: 35  (no try/catch successors)
no phis
 #187:remove;0:remove: -1: Assume               v49 <- v54; not null
 #188:remove;0:remove: -1: ConstNumber          v32(0b0) <-  0 (INT)
                     : -1: ConstNumber          v33(0) <-  0 (@Null NULL)
                     : -1: Goto                 block 35

block 35, pred-counts: 1, succ-count: 0, filled: true, sealed: false
predecessors: 34 
successors: -
no phis
 #188:remove;0:remove: -1: ConstString          v90 <-  "key"
                     : -1: Invoke-Static        v45, v90; method: void kotlin.jvm.internal.Intrinsics.checkNotNullParameter(java.lang.Object, java.lang.String)
                     : -1: ConstString          v91 <-  "oldValue"
                     : -1: Invoke-Static        v49, v91; method: void kotlin.jvm.internal.Intrinsics.checkNotNullParameter(java.lang.Object, java.lang.String)
 #191:remove;0:remove: -1: Return               v49

block 37, pred-counts: 1, succ-count: 0, filled: true, sealed: false
predecessors: 33 
successors: -
no phis
 #191:remove;0:remove: -1: Return               v54

After dead code removal a single return is left with the non null assumption:

block 34, pred-counts: 1, succ-count: 1, filled: true, sealed: false
predecessors: 32 
successors: 35  (no try/catch successors)
no phis
 #187:remove;0:remove: -1: Assume               v49 <- v54; not null
 #188:remove;0:remove: -1: Goto                 block 35

block 35, pred-counts: 1, succ-count: 0, filled: true, sealed: false
predecessors: 34 
successors: -
no phis
 #191:remove;0:remove: -1: Return               v49

For reference the source of the method is:

    public fun remove(key: K): V? {
        val previous: V?
        lock.synchronized {
            previous = map.remove(key)
            if (previous != null) {
                size -= safeSizeOf(key, previous)
            }
        }

        if (previous != null) {
            entryRemoved(false, key, previous, null)
        }

        return previous
    }

with the default implementation of entryRemoved doing nothing:

    protected open fun entryRemoved(evicted: Boolean, key: K, oldValue: V, newValue: V?) {}

The underlying issue is that branch simplification (as part of dead code removal) is comparing return values through their aliased values (in valuesAreIdentical) potentially conflating return values with different assume instructions. As this happens before collecting optimization info this can lead to the wrong dynamic type for the return value.

  private boolean valuesAreIdentical(Value value, Value other) {
    value = value.getAliasedValue();
    other = other.getAliasedValue();
    if (value == other) {
      return true;
    }
    if (value.isPhi() || other.isPhi()) {
      return false;
    }
    return instructionsDefineIdenticalValues(value.definition, other.definition);
  }

Project: r8
Branch: main
Author: Søren Gjesse <sgjesse@google.com>
Link:      https://r8-review.googlesource.com/101409

Add reproduction of b/395489597

Add reproduction of b/395489597 
 
Bug: b/395489597 
Change-Id: If769fae5db93ee8e2cfc1cf9feeddd847b00832b