# HG changeset patch # User Doug Simon # Date 1389218393 -3600 # Node ID 4fc8c8bb4c32878cc04b064d2ac9ad1fce1a85e0 # Parent 5252d833945d022bb03b5601dccf2aeeeba8eaf0 removed Assembler::no_operand alias diff -r 5252d833945d -r 4fc8c8bb4c32 src/cpu/x86/vm/assembler_x86.hpp --- a/src/cpu/x86/vm/assembler_x86.hpp Wed Jan 08 22:59:24 2014 +0100 +++ b/src/cpu/x86/vm/assembler_x86.hpp Wed Jan 08 22:59:53 2014 +0100 @@ -527,9 +527,6 @@ enum WhichOperand { // input to locate_operand, and format code for relocations imm_operand = 0, // embedded 32-bit|64-bit immediate operand -#ifdef GRAAL - no_operand = imm_operand, // format value for relocations at instructions with no operand -#endif disp32_operand = 1, // embedded 32-bit displacement or address call32_operand = 2, // embedded 32-bit self-relative displacement #ifndef _LP64 diff -r 5252d833945d -r 4fc8c8bb4c32 src/cpu/x86/vm/graalCodeInstaller_x86.hpp --- a/src/cpu/x86/vm/graalCodeInstaller_x86.hpp Wed Jan 08 22:59:24 2014 +0100 +++ b/src/cpu/x86/vm/graalCodeInstaller_x86.hpp Wed Jan 08 22:59:53 2014 +0100 @@ -247,8 +247,11 @@ break; } case MARK_POLL_FAR: - // This is a load from a register so there is no relocatable operand - _instructions->relocate(pc, relocInfo::poll_type, Assembler::no_operand); + // This is a load from a register so there is no relocatable operand. + // We just have to ensure that the format is not disp32_operand + // so that poll_Relocation::fix_relocation_after_move does the right + // thing (i.e. ignores this relocation record) + _instructions->relocate(pc, relocInfo::poll_type, Assembler::imm_operand); break; case MARK_POLL_RETURN_NEAR: { relocate_poll_near(pc); @@ -256,8 +259,8 @@ break; } case MARK_POLL_RETURN_FAR: - // This is a load from a register so there is no relocatable operand - _instructions->relocate(pc, relocInfo::poll_return_type, Assembler::no_operand); + // see comment above for MARK_POLL_FAR + _instructions->relocate(pc, relocInfo::poll_return_type, Assembler::imm_operand); break; default: fatal("invalid mark value");