RFR(S): 8193927: Optimize scanning code for oops.

[Doerr, Martin]

Hi Götz,

this looks good, now.

Thanks and best regards,
Martin


-----Original Message-----
From: Lindenmaier, Goetz 
Sent: Dienstag, 2. Januar 2018 15:21
To: Andrew Haley <aph at redhat.com>; Doerr, Martin <martin.doerr at sap.com>; hotspot-compiler-dev at openjdk.java.net
Subject: RE: RFR(S): 8193927: Optimize scanning code for oops.

Hi,

I please need a sponsor.

I changed the flag on both arm platforms and edited the comment:
http://cr.openjdk.java.net/~goetz/wr17/8193927-oopsInCode/webrev.03/

Best regards,
  Goetz.

> -----Original Message-----
> From: Andrew Haley [mailto:aph at redhat.com]
> Sent: Dienstag, 2. Januar 2018 14:39
> To: Doerr, Martin <martin.doerr at sap.com>; Lindenmaier, Goetz
> <goetz.lindenmaier at sap.com>; hotspot-compiler-dev at openjdk.java.net
> Subject: Re: RFR(S): 8193927: Optimize scanning code for oops.
> 
> On 02/01/18 10:48, Doerr, Martin wrote:
> 
> > If I understand it correctly, only usages of
> > oop_Relocation::spec_for_immediate() need it which uses oop_index =
> > 0. Other oop_Relocations use the CodeBlob's oop pool and the oops
> > will be found there by iterating over
> > "oop* p = oops_begin(); p < oops_end(); p++".
> 
> Ah, OK.  We don't use oop_Relocation::spec_for_immediate(), so you're
> right, this is x86 only.
> 
> --
> Andrew Haley
> Java Platform Lead Engineer
> Red Hat UK Ltd. <https://www.redhat.com>
> EAC8 43EB D3EF DB98 CC77 2FAD A5CD 6035 332F A671