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