RFR: 8028739: javac generates incorrect descriptor for MethodHandle::invoke

[Robert Field]

Thanks John!

Updated webrev:

http://cr.openjdk.java.net/~rfield/8028739v1/ 
<http://cr.openjdk.java.net/%7Erfield/8028739v1/>

Only change is to use the pre-existing test for signature polymorphic:

2121             final boolean isSignaturePolymorphic() {
2122                 return  tree.sym instanceof MethodSymbol &&
2123                         types.isSignaturePolymorphic((MethodSymbol)tree.sym);
2124             }


-Robert


On 11/21/13 22:55, John Rose wrote:
> I think you will get a false positive on MethodHandle.invokeWithArguments.
>
> There are more precise tests for sig. polymorphism. It must be native final; the code should check for that.
>
> -- John  (on my iPhone)
>
> On Nov 21, 2013, at 8:21 PM, Robert Field <robert.field at oracle.com> wrote:
>
>> Please review fix for:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8028739
>>
>> Webrev:
>>
>> http://cr.openjdk.java.net/~rfield/8028739v0/
>> <http://cr.openjdk.java.net/%7Erfield/8028739v0/>
>>
>> Thank you,
>> Robert
>>
>>