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

[Lindenmaier, Goetz]

Hi Vladimir, 

thanks for sponsoring!

Best regards,
  Goetz.

> -----Original Message-----
> From: Vladimir Kozlov [mailto:vladimir.kozlov at oracle.com]
> Sent: Mittwoch, 3. Januar 2018 00:45
> To: Lindenmaier, Goetz <goetz.lindenmaier at sap.com>; 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 Goetz,
> 
> I started testing. Will push it if testing pass.
> 
> Regards,
> Vladimir
> 
> On 1/2/18 6:20 AM, Lindenmaier, Goetz wrote:
> > 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