Clean up ClassLinker::VerifyClass.
And drop kAccWasVerificationAttempted. It's not needed and duplicates
logic already present in the compiler and the class status.
This CL also enables nterp running methods with soft failures.
Test: test.py
Bug: 28313047
Change-Id: I853a6f00b9e0c38091d86fcd77167c92ff5b383c
diff --git a/runtime/class_linker_test.cc b/runtime/class_linker_test.cc
index d1f3a17..e0f498d 100644
--- a/runtime/class_linker_test.cc
+++ b/runtime/class_linker_test.cc
@@ -116,8 +116,7 @@
EXPECT_EQ(0, primitive->GetIfTableCount());
EXPECT_TRUE(primitive->GetIfTable() != nullptr);
EXPECT_EQ(primitive->GetIfTable()->Count(), 0u);
- EXPECT_EQ(kAccPublic | kAccFinal | kAccAbstract | kAccVerificationAttempted,
- primitive->GetAccessFlags());
+ EXPECT_EQ(kAccPublic | kAccFinal | kAccAbstract, primitive->GetAccessFlags());
}
void AssertObjectClass(ObjPtr<mirror::Class> JavaLangObject)
@@ -1446,8 +1445,6 @@
static void CheckVerificationAttempted(ObjPtr<mirror::Class> c, bool preverified)
REQUIRES_SHARED(Locks::mutator_lock_) {
- EXPECT_EQ((c->GetAccessFlags() & kAccVerificationAttempted) != 0U, preverified)
- << "Class " << mirror::Class::PrettyClass(c) << " not as expected";
for (auto& m : c->GetMethods(kRuntimePointerSize)) {
CheckMethod(&m, preverified);
}