8231953: Wrong assumption in assertion in oop::register_oop
authorcoleenp
Thu, 24 Oct 2019 08:49:21 -0400
changeset 58782 10d2642d1905
parent 58781 d6206dda2e28
child 58783 d55b62dbe688
8231953: Wrong assumption in assertion in oop::register_oop Summary: On ARM32 thumb mode, the pc of the current frame is always zero Reviewed-by: coleenp, dcubed Contributed-by: christoph.goettschkes@microdoc.com
src/hotspot/share/oops/oopsHierarchy.cpp
--- a/src/hotspot/share/oops/oopsHierarchy.cpp	Thu Oct 24 15:46:50 2019 +0300
+++ b/src/hotspot/share/oops/oopsHierarchy.cpp	Thu Oct 24 08:49:21 2019 -0400
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 2005, 2015, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 2005, 2019, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -37,8 +37,6 @@
   Thread* t = Thread::current_or_null();
   if (t != NULL && t->is_Java_thread()) {
      frame fr = os::current_frame();
-     // This points to the oop creator, I guess current frame points to caller
-     assert (fr.pc(), "should point to a vm frame");
      t->unhandled_oops()->register_unhandled_oop(this, fr.pc());
   }
 }