Revert^2 "Add verifier fallback for JVMTI Get/SetLocalVariable"
This reverts commit 99cbfb55fc7ac0f65b1ccdc7076219fcee383b92.
This unreverts commit e48fd0b4780efadc6b3433fe7a56aa5be2a84325.
We were incorrectly bounds-checking the register number. We were
treating it as unsigned when it was actual signed. Previously this
wouldn't matter since normally the debug-info won't have any
information for negative slots but by falling back to the verifier
we hit some check failures.
Reason for revert: Fixed underlying issue with bad bounds check.
Bug: 131711256
Change-Id: I0b859ce322f3b23f937b72d735db8f6870c40602
diff --git a/test/knownfailures.json b/test/knownfailures.json
index 963d335..ad0cbe8 100644
--- a/test/knownfailures.json
+++ b/test/knownfailures.json
@@ -777,6 +777,18 @@
"description": ["Requires zip, which isn't available on device"]
},
{
+ "tests": [
+ "1965-get-set-local-primitive-no-tables",
+ "1966-get-set-local-objects-no-table"
+ ],
+ "variant": "jvm",
+ "bug": "133241695",
+ "description": [
+ "The RI is wildly inconsistent about how it handles Get/SetLocalVariable when classes ",
+ "lack debug info."
+ ]
+ },
+ {
"tests": ["683-clinit-inline-static-invoke"],
"variant": "jvm",
"description": ["Uses android-specific boot image class."]