Mercurial > hg > graal-compiler
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);