Remove asserts in XArrayKlass::oop_oop_iterate()

rkennke · Sep 10, 2024 · 6b27782 · 6b27782
1 parent 321a44a
commit 6b27782
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 4 deletions.
diff --git a/src/hotspot/share/oops/objArrayKlass.inline.hpp b/src/hotspot/share/oops/objArrayKlass.inline.hpp
@@ -70,8 +70,7 @@ void ObjArrayKlass::oop_oop_iterate_elements_bounded(
 
 template <typename T, typename OopClosureType>
 void ObjArrayKlass::oop_oop_iterate(oop obj, OopClosureType* closure) {
-  // In this assert, we cannot safely access the Klass* with compact headers.
-  assert (UseCompactObjectHeaders || obj->is_array(), "obj must be array");
+  assert(obj->is_array(), "obj must be array");
   objArrayOop a = objArrayOop(obj);
 
   if (Devirtualizer::do_metadata(closure)) {

diff --git a/src/hotspot/share/oops/typeArrayKlass.inline.hpp b/src/hotspot/share/oops/typeArrayKlass.inline.hpp
@@ -35,8 +35,7 @@
 class OopIterateClosure;
 
 inline void TypeArrayKlass::oop_oop_iterate_impl(oop obj, OopIterateClosure* closure) {
-  // In this assert, we cannot safely access the Klass* with compact headers.
-  assert(UseCompactObjectHeaders || obj->is_typeArray(),"must be a type array");
+  assert(obj->is_typeArray(),"must be a type array");
   // Performance tweak: We skip processing the klass pointer since all
   // TypeArrayKlasses are guaranteed processed via the null class loader.
 }