RFR: 8165492: Reduce number of lambda forms generated by MethodHandleInlineCopyStrategy
Aleksey Shipilev
shade at redhat.com
Thu Sep 8 18:56:26 UTC 2016
Hi,
On 09/08/2016 08:03 PM, Claes Redestad wrote:
> Webrev: http://cr.openjdk.java.net/~redestad/8165492/webrev.01/
It is a bit sad that we have to bust the doors with internal APIs, and
not use the public API, thus robbing ourselves of the opportunity to
optimize the public API for a wide range of use cases.
Barring that, the patch looks good.
> On 09/08/2016 09:21 PM, Paul Sandoz wrote:
> Did you consider replacing the if block with an assert? presumably if
> it is non-zero it is an internal error?
The original check was to guard against Unsafe.allocateUnitializedArray
usage, and so I would like to keep the code that checks the index
unconditionally. asserts can be disabled.
Thanks,
-Aleksey
More information about the core-libs-dev
mailing list