[lworld] RFR: JDK-8247663: [lworld] LambdaMetaFactory should allow an inline value projection type to convert to its reference projection type [v2]

Mandy Chung mchung at openjdk.java.net
Wed Jul 8 20:26:59 UTC 2020


On Wed, 8 Jul 2020 20:09:04 GMT, Mandy Chung <mchung at openjdk.org> wrote:

>> src/java.base/share/classes/java/lang/invoke/AbstractValidatingLambdaMetafactory.java line 402:
>> 
>>> 401:         }
>>> 402:
>>> 403:         if (!fromType.isInlineClass() && !toType.isInlineClass()) {
>> 
>> Since fromType.valueType is an Optional<Class<?>> the code could be rearranged to use is Present() and not double check
>> for inline so many times. If it is a performance issue it should be refactored to do the complete check of one case
>> before the other.
>
> I simplified the check.  I also uncovered a bug in the spec for `Class::valueType` and `Class::referenceType` and
> included a spec fix in this patch.

@RogerRiggs can you check out the delta patch:
   https://github.com/openjdk/valhalla/pull/102/commits/b295ac571e78c12e8bb80a4568aab0d79cdd3d06

-------------

PR: https://git.openjdk.java.net/valhalla/pull/102



More information about the valhalla-dev mailing list