summaryrefslogtreecommitdiff
path: root/test/MultiDexModifiedSecondary
diff options
context:
space:
mode:
authorVladimir Marko <vmarko@google.com>2017-09-15 13:30:06 +0100
committerVladimir Marko <vmarko@google.com>2017-09-15 17:17:44 +0100
commitaa0912c4e91759441ce717f4f2089078e0e2a3d1 (patch)
tree4777dcbb7bd5ded78408d487f9302ab73297b02a /test/MultiDexModifiedSecondary
parentd0e4c9d1137f0680307f5c1b630008aacdcf3da8 (diff)
Do not fill DexCache methods in resolution trampoline.
Storing the methods in DexCache was done for performance reasons back when compiled code used the DexCache for method calls. HInvokeStaticOrDirect does not use the DexCache since replacing the kDexCachePcRelative with kBssEntry in https://android-review.googlesource.com/399312 , so we do not need to populate the DexCache for performance reasons. The storing of the ArtMethod* for invoke-super was also erroneous as it may have stored a method in a slot that references a class not present in the associated ClassTable. This led to crashes when subsequently trying to resolve the method and check for ICCE from JIT. Therefore we remove the code that stores the method, both for the crashing invoke-super case and the obsolete invoke-static case (introduced due to bug 19175856). Test: 164-resolution-trampoline-dex-cache Test: m test-art-host-gtest Test: testrunner.py --host Bug: 64759619 Change-Id: Ieee68d3c6a731e61bc8115085c1e027af199fc59
Diffstat (limited to 'test/MultiDexModifiedSecondary')
0 files changed, 0 insertions, 0 deletions