# HG changeset patch # User Thomas Wuerthinger # Date 1396717867 -7200 # Node ID 82ff6c8c8f6eb19d02b84cc548ac2e55b1354582 # Parent 3b318ba935d5a7f833e162771422cbb7e0a3c3d7 Increase maximum allocatable spill area by factor of 2. diff -r 3b318ba935d5 -r 82ff6c8c8f6e src/share/vm/compiler/oopMap.cpp --- a/src/share/vm/compiler/oopMap.cpp Sat Apr 05 03:24:04 2014 +0200 +++ b/src/share/vm/compiler/oopMap.cpp Sat Apr 05 19:11:07 2014 +0200 @@ -176,8 +176,6 @@ void OopMap::set_value(VMReg reg) { // At this time, we only need value entries in our OopMap when ZapDeadCompiledLocals is active. - if (ZapDeadCompiledLocals) - set_xxx(reg, OopMapValue::value_value, VMRegImpl::Bad()); } @@ -409,7 +407,7 @@ } // We want coop, value and oop oop_types - int mask = OopMapValue::oop_value | OopMapValue::value_value | OopMapValue::narrowoop_value; + int mask = OopMapValue::oop_value | OopMapValue::narrowoop_value; { for (OopMapStream oms(map,mask); !oms.is_done(); oms.next()) { omv = oms.current(); @@ -440,10 +438,6 @@ } #endif // ASSERT oop_fn->do_oop(loc); - } else if ( omv.type() == OopMapValue::value_value ) { - assert((*loc) == (oop)NULL || !Universe::is_narrow_oop_base(*loc), - "found invalid value pointer"); - value_fn->do_oop(loc); } else if ( omv.type() == OopMapValue::narrowoop_value ) { narrowOop *nl = (narrowOop*)loc; #ifndef VM_LITTLE_ENDIAN @@ -544,9 +538,6 @@ case OopMapValue::oop_value: st->print("Oop"); break; - case OopMapValue::value_value: - st->print("Value"); - break; case OopMapValue::narrowoop_value: st->print("NarrowOop"); break; diff -r 3b318ba935d5 -r 82ff6c8c8f6e src/share/vm/compiler/oopMap.hpp --- a/src/share/vm/compiler/oopMap.hpp Sat Apr 05 03:24:04 2014 +0200 +++ b/src/share/vm/compiler/oopMap.hpp Sat Apr 05 19:11:07 2014 +0200 @@ -54,7 +54,7 @@ public: // Constants - enum { type_bits = 5, + enum { type_bits = 4, register_bits = BitsPerShort - type_bits }; enum { type_shift = 0, @@ -68,10 +68,9 @@ enum oop_types { // must fit in type_bits unused_value =0, // powers of 2, for masking OopMapStream oop_value = 1, - value_value = 2, - narrowoop_value = 4, - callee_saved_value = 8, - derived_oop_value= 16 }; + narrowoop_value = 2, + callee_saved_value = 4, + derived_oop_value= 8 }; // Constructors OopMapValue () { set_value(0); set_content_reg(VMRegImpl::Bad()); } @@ -96,13 +95,11 @@ // Querying bool is_oop() { return mask_bits(value(), type_mask_in_place) == oop_value; } - bool is_value() { return mask_bits(value(), type_mask_in_place) == value_value; } bool is_narrowoop() { return mask_bits(value(), type_mask_in_place) == narrowoop_value; } bool is_callee_saved() { return mask_bits(value(), type_mask_in_place) == callee_saved_value; } bool is_derived_oop() { return mask_bits(value(), type_mask_in_place) == derived_oop_value; } void set_oop() { set_value((value() & register_mask_in_place) | oop_value); } - void set_value() { set_value((value() & register_mask_in_place) | value_value); } void set_narrowoop() { set_value((value() & register_mask_in_place) | narrowoop_value); } void set_callee_saved() { set_value((value() & register_mask_in_place) | callee_saved_value); } void set_derived_oop() { set_value((value() & register_mask_in_place) | derived_oop_value); } diff -r 3b318ba935d5 -r 82ff6c8c8f6e src/share/vm/runtime/vmStructs.cpp --- a/src/share/vm/runtime/vmStructs.cpp Sat Apr 05 03:24:04 2014 +0200 +++ b/src/share/vm/runtime/vmStructs.cpp Sat Apr 05 19:11:07 2014 +0200 @@ -2646,7 +2646,6 @@ declare_constant(OopMapValue::register_mask_in_place) \ declare_constant(OopMapValue::unused_value) \ declare_constant(OopMapValue::oop_value) \ - declare_constant(OopMapValue::value_value) \ declare_constant(OopMapValue::narrowoop_value) \ declare_constant(OopMapValue::callee_saved_value) \ declare_constant(OopMapValue::derived_oop_value) \