Review request for JDK-8062401: User accessors require boxing and do not support optimistic types

[Hannes Wallnoefer]

Thanks Marcus.

All box2d user accessors dealing with primitive values now are fully 
primitive and optimistic. There is a slight improvement in performance, 
but I'm not sure it's the best we can get. I'll contact Vladimir about 
how to find out.

I'll also add a comment to the benchmark as you suggested.

Hannes

Am 2014-10-30 um 11:43 schrieb Marcus Lagergren:
> +1
>
> What are the performance implications? Have you done some octane runs or something?
>
> Furthermore, I’d like to see a comment in the benchmark you’ve added, what it’s for, a reference to the issue number JDK-8062401 in a comment and what the difference before and after was.
>
> Otherwise - solid work.  Nicely done.
>
> Have you discussed with Vladimir Ivanov, the problem he had with box2d and the inlining site in UserAccessorProperty? Will this affect it and make it better? CC:ing Vladimir.
>
> Regards
> Marcus
>
>> On 29 Oct 2014, at 16:41, Hannes Wallnoefer <hannes.wallnoefer at oracle.com> wrote:
>>
>> Please review JDK-8062401: User accessors require boxing and do not support optimistic types:
>>
>> http://cr.openjdk.java.net/~hannesw/8062401/
>>
>> Thanks,
>> Hannes