# HG changeset patch # User coleenp # Date 1571921361 14400 # Node ID 10d2642d1905d1f3c35019d7de123861ec88375d # Parent d6206dda2e2887cb5aa1ae37f542985a6430a191 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 diff -r d6206dda2e28 -r 10d2642d1905 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()); } }