6798785: Crash in OopFlow::build_oop_map: incorrect comparison of 64bit pointers
authornever
Wed, 04 Feb 2009 11:44:57 -0800
changeset 2026 d15da7324f58
parent 2025 a13c4b3f024e
child 2027 a980915cf495
6798785: Crash in OopFlow::build_oop_map: incorrect comparison of 64bit pointers Reviewed-by: phh, kvn
hotspot/src/share/vm/adlc/dict2.cpp
hotspot/src/share/vm/libadt/dict.cpp
--- a/hotspot/src/share/vm/adlc/dict2.cpp	Tue Feb 03 18:05:19 2009 -0800
+++ b/hotspot/src/share/vm/adlc/dict2.cpp	Wed Feb 04 11:44:57 2009 -0800
@@ -316,9 +316,12 @@
   return strcmp((const char *)k1,(const char *)k2);
 }
 
-// Slimey cheap key comparator.
+// Cheap key comparator.
 int cmpkey(const void *key1, const void *key2) {
-  return (int)((intptr_t)key1 - (intptr_t)key2);
+  if (key1 == key2) return 0;
+  intptr_t delta = (intptr_t)key1 - (intptr_t)key2;
+  if (delta > 0) return 1;
+  return -1;
 }
 
 //=============================================================================
--- a/hotspot/src/share/vm/libadt/dict.cpp	Tue Feb 03 18:05:19 2009 -0800
+++ b/hotspot/src/share/vm/libadt/dict.cpp	Wed Feb 04 11:44:57 2009 -0800
@@ -346,9 +346,12 @@
   return strcmp((const char *)k1,(const char *)k2);
 }
 
-// Slimey cheap key comparator.
+// Cheap key comparator.
 int32 cmpkey(const void *key1, const void *key2) {
-  return (int32)((intptr_t)key1 - (intptr_t)key2);
+  if (key1 == key2) return 0;
+  intptr_t delta = (intptr_t)key1 - (intptr_t)key2;
+  if (delta > 0) return 1;
+  return -1;
 }
 
 //=============================================================================