Loosen class status check after resolving.
If `ClassLinker::LinkClass()` creates a new `Class` object,
it takes an `ObjectLock` on that object but releases that
lock when it returns to the `ClassLinker::DefineClass()`.
Therefore another thread can try and fail to verify the
class and mark it as resolved erroneous. The `CHECK()` that
the class is not resolved erroneous was therefore wrong
and we remove that part.
Also dump the class status if the remaining `CHECK()` fails.
Test: m test-art-host-gtest
Test: testrunner.py --host --optimizing
Bug: 202091046
Change-Id: Ida9c0d45478fbae913fcaabcc237048d28a4dd3d
diff --git a/runtime/class_linker.cc b/runtime/class_linker.cc
index c85faf5..df50387 100644
--- a/runtime/class_linker.cc
+++ b/runtime/class_linker.cc
@@ -3197,7 +3197,7 @@
}
self->AssertNoPendingException();
CHECK(h_new_class != nullptr) << descriptor;
- CHECK(h_new_class->IsResolved() && !h_new_class->IsErroneousResolved()) << descriptor;
+ CHECK(h_new_class->IsResolved()) << descriptor << " " << h_new_class->GetStatus();
// Instrumentation may have updated entrypoints for all methods of all
// classes. However it could not update methods of this class while we