possible bug in x86 MacroAssembler::delayed_value_impl

Christian Thalinger Christian.Thalinger at Sun.COM
Fri May 15 05:30:41 PDT 2009


Hi!

While looking at the code I found this one:

diff --git a/src/cpu/x86/vm/assembler_x86.cpp b/src/cpu/x86/vm/assembler_x86.cpp
--- a/src/cpu/x86/vm/assembler_x86.cpp
+++ b/src/cpu/x86/vm/assembler_x86.cpp
@@ -7644,7 +7644,7 @@ RegisterOrConstant MacroAssembler::delay
 
 #ifdef ASSERT
   Label L;
-  testl(tmp, tmp);
+  testptr(tmp, tmp);
   jccb(Assembler::notZero, L);
   hlt();
   bind(L);

I think it's a bug since tmp is a pointer.  Maybe an overlook during
32/64-bit merging?

-- Christian




More information about the hotspot-compiler-dev mailing list