Fix assertion failure in b15

[Andrew Haley]

b15 triggers assertion failures in the JIT with an AMD64 debug build.

 > openjdk/control/build/linux-amd64-debug/tmp/java/java.lang/.classes.list
 > # To suppress the following error report, specify this argument
 > # after -XX: or in .hotspotrc:  SuppressErrorAt=/assembler_amd64.cpp:663
 > #
 > # An unexpected error has been detected by Java Runtime Environment:
 > #
 > #  Internal Error (/local/icedtea/openjdk/hotspot/src/cpu/amd64/vm/assembler_amd64.cpp:663), pid=6568, tid=1077938496
 > #  Error: assert(which == imm64_operand && is_64bit,"")
 > #

If the operand of a call instruction won't fit inside 32 bits, the JIT
generates code that looks like:

      move addr64, %r10
      call *%r10

The assertion logic that checks the JIT doesn't know about this
possibility, and assumes that all call instructions have 32-bit
operands.  It's a trivial matter to add the 64-bit immediate case.

I've added this patch to IcedTea.  Which list should I post this to?

Andrew.




2007-07-11  Andrew Haley  <aph at redhat.com>

	* openjdk/hotspot/src/cpu/amd64/vm/assembler_amd64.cpp 
	(Assembler::check_relocation): Add 64-bit calls.

*** openjdk/hotspot/src/cpu/amd64/vm/assembler_amd64.cpp~	2007-07-05 08:16:04.000000000 +0100
--- openjdk/hotspot/src/cpu/amd64/vm/assembler_amd64.cpp	2007-07-11 17:33:58.000000000 +0100
*************** void Assembler::check_relocation(Relocat
*** 884,887 ****
--- 884,889 ----
    if (r->type() == relocInfo::none) {
      return;
+   } else if (r->is_call() && format == imm64_operand) {
+     opnd = locate_operand(inst, imm64_operand);
    } else if (r->is_call() || format == call32_operand) {
      opnd = locate_operand(inst, call32_operand);

-- 
Red Hat UK Ltd, Amberley Place, 107-111 Peascod Street, Windsor, Berkshire, SL4 1TE, UK
Registered in England and Wales No. 3798903