RFR: 8173472: AArch64: C1 comparisons with null only use 32-bit instructions
Andrew Haley
aph at redhat.com
Mon Jan 30 11:26:18 UTC 2017
On 30/01/17 01:06, David Holmes wrote:
> IIRC C1 was 32-bit only - maybe even never officially supported on
> 64-bit, just as part of tiered ??
Um, what? Officially supported or not, it's still gotta work.
> Is this code buggy or just quirky?
Beats me. I'm not sure that T_ADDRESS can ever happen anyway in
that context. You'll see exactly the same in x86:
case T_ADDRESS: {
assert(patch_code == lir_patch_none, "no patching handled here");
__ movptr(dest->as_register(), c->as_jint());
break;
}
Perhaps this should be looked at, but it's not urgent.
Andrew.
More information about the hotspot-dev
mailing list