8080627: JavaThread::satb_mark_queue_offset() is too big for an ARM ldrsb instruction
authorbrutisso
Fri, 22 May 2015 10:56:37 +0200
changeset 30872 eccf4a34928e
parent 30871 e90a8de769e4
child 30877 dae0933ba77a
8080627: JavaThread::satb_mark_queue_offset() is too big for an ARM ldrsb instruction Reviewed-by: roland, kbarrett
hotspot/src/share/vm/c1/c1_LIRGenerator.cpp
--- a/hotspot/src/share/vm/c1/c1_LIRGenerator.cpp	Thu May 21 09:35:59 2015 +0200
+++ b/hotspot/src/share/vm/c1/c1_LIRGenerator.cpp	Fri May 22 10:56:37 2015 +0200
@@ -1469,7 +1469,9 @@
   } else {
     guarantee(in_bytes(PtrQueue::byte_width_of_active()) == 1,
               "Assumption");
-    flag_type = T_BYTE;
+    // Use unsigned type T_BOOLEAN here rather than signed T_BYTE since some platforms, eg. ARM,
+    // need to use unsigned instructions to use the large offset to load the satb_mark_queue.
+    flag_type = T_BOOLEAN;
   }
   LIR_Opr thrd = getThreadPointer();
   LIR_Address* mark_active_flag_addr =