changeset 17428:50942f016967

remove duplicate check in OopRecorder
author Tom Rodriguez <tom.rodriguez@oracle.com>
date Mon, 13 Oct 2014 15:52:19 -0700
parents 307b26f8b5d1
children f609dff05ea0
files src/share/vm/code/oopRecorder.cpp src/share/vm/code/oopRecorder.hpp
diffstat 2 files changed, 1 insertions(+), 14 deletions(-) [+]
line wrap: on
line diff
--- a/src/share/vm/code/oopRecorder.cpp	Mon Oct 13 21:15:03 2014 +0200
+++ b/src/share/vm/code/oopRecorder.cpp	Mon Oct 13 15:52:19 2014 -0700
@@ -34,15 +34,6 @@
 template <class T> int ValueRecorder<T>::_find_index_calls = 0;
 template <class T> int ValueRecorder<T>::_hit_indexes      = 0;
 template <class T> int ValueRecorder<T>::_missed_indexes   = 0;
-
-void OopRecorder::check_for_duplicates(int index, jobject h) {
-  oop o = JNIHandles::resolve(h);
-  for (int i = 1; i < oop_count(); i++) {
-    if (o == JNIHandles::resolve(oop_at(i)) && index != i) {
-      assert(false, "duplicate found");
-    }
-  }
-}
 #endif //ASSERT
 
 template <class T> ValueRecorder<T>::ValueRecorder(Arena* arena) {
--- a/src/share/vm/code/oopRecorder.hpp	Mon Oct 13 21:15:03 2014 +0200
+++ b/src/share/vm/code/oopRecorder.hpp	Mon Oct 13 15:52:19 2014 -0700
@@ -192,15 +192,11 @@
     }
   }
 
-  void check_for_duplicates(int index, jobject h) NOT_DEBUG_RETURN;
-
   int allocate_oop_index(jobject h) {
     return _oops.allocate_index(h);
   }
   int find_index(jobject h) {
-    int result = _object_lookup != NULL ? _object_lookup->find_index(h, this) : _oops.find_index(h);
-    check_for_duplicates(result, h);
-    return result;
+    return _object_lookup != NULL ? _object_lookup->find_index(h, this) : _oops.find_index(h);
   }
   jobject oop_at(int index) {
     return _oops.at(index);