--- old/src/hotspot/share/gc/shared/barrierSet.hpp 2019-09-10 12:46:02.377145104 +0200 +++ new/src/hotspot/share/gc/shared/barrierSet.hpp 2019-09-10 12:46:02.127136844 +0200 @@ -144,10 +144,6 @@ virtual void make_parsable(JavaThread* thread) {} -#ifdef CHECK_UNHANDLED_OOPS - virtual bool oop_equals_operator_allowed() { return true; } -#endif - public: // Print a description of the memory for the barrier set virtual void print_on(outputStream* st) const = 0; --- old/src/hotspot/share/oops/oopsHierarchy.cpp 2019-09-10 12:46:02.749157393 +0200 +++ new/src/hotspot/share/oops/oopsHierarchy.cpp 2019-09-10 12:46:02.490148837 +0200 @@ -23,9 +23,6 @@ */ #include "precompiled.hpp" -#include "gc/shared/barrierSet.hpp" -#include "gc/shared/collectedHeap.hpp" -#include "gc/shared/collectedHeap.inline.hpp" #include "memory/universe.hpp" #include "oops/oopsHierarchy.hpp" #include "runtime/thread.inline.hpp" @@ -56,14 +53,4 @@ } } -bool oop::operator==(const oop o) const { - assert(BarrierSet::barrier_set()->oop_equals_operator_allowed(), "Not allowed"); - return obj() == o.obj(); -} - -bool oop::operator!=(const volatile oop o) const { - assert(BarrierSet::barrier_set()->oop_equals_operator_allowed(), "Not allowed"); - return obj() != o.obj(); -} - #endif // CHECK_UNHANDLED_OOPS --- old/src/hotspot/share/oops/oopsHierarchy.hpp 2019-09-10 12:46:03.114169452 +0200 +++ new/src/hotspot/share/oops/oopsHierarchy.hpp 2019-09-10 12:46:02.855160895 +0200 @@ -100,9 +100,9 @@ // General access oopDesc* operator->() const { return obj(); } - bool operator==(const oop o) const; + bool operator==(const oop o) const { return obj() == o.obj(); } bool operator==(void *p) const { return obj() == p; } - bool operator!=(const volatile oop o) const; + bool operator!=(const volatile oop o) const { return obj() != o.obj(); } bool operator!=(void *p) const { return obj() != p; } // Assignment