[OpenJDK 2D-Dev] Java2d code using @ForceInline or @Stable annotations

[Philip Race]

A demonstrable 5-10% on a "real program" - one that does something that
is at least representative of a real world benefit is something
you could (should) take to the hotspot team (probably hotspot-compiler-dev)
and point them directly at the program and the results and see what they 
say.
Who knows .. they could say 325 is the result of lots of careful analysis
or they may make some changes as a result, perhaps not to that metric
but to something else.

-phil.



On 3/16/16, 3:25 PM, Laurent Bourgès wrote:
>
> Phil,
>
> I tested adding @ForceInline to few important methods in marlin that 
> jitwatch detected as too large (>325 byte codes).
>
> It seems hotspot perform inlining with the hint and few long tests are 
> a bit faster (max 10%).
>
> I agree it is border line and java2d must respect vm and module rules.
>
> Cheers,
> Laurent
>
> Le 16 mars 2016 17:51, "Philip Race" <philip.race at oracle.com 
> <mailto:philip.race at oracle.com>> a écrit :
> >
> > The jigsaw team would need to sign off on any changes
> > to the module graph and in addition I think the VM team
> > might have some input on this since I don't think this
> > is intended to be used other than extremely sparingly.
> > I see @Stable used only by String and java.lang.invoke
> > which has deep ties to the VM.
> > In other words I am not sure we should do this or if, for ForceIine,
> > it would bring any benefits over what hotspot is doing without 
> prompting.
> >
> > -phil.
> >
> >
> > On 3/16/16, 8:24 AM, Laurent Bourgès wrote:
> >>
> >> Hi,
> >>
> >> In the marlin renderer, I am wondering if I could use the compiler 
> annotations @ForceInline / @Stable (in java.base module).
> >>
> >> It would provide hints to the hotspot compiler and maybe bring some 
> performance benefits.
> >>
> >> Of course, it means adding a qualified export in the modules.xml:
> >> <export>
> >> <name>jdk.internal.vm.annotation</name>
> >> <to>java.desktop</to>
> >> </export>
> >>
> >> Please give me your opinion & comments.
> >>
> >> Laurent
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/2d-dev/attachments/20160316/27832969/attachment.html>