Request for review (M): 6914206: change way of permission checking for generated MethodHandle adapters

John Rose John.Rose at Sun.COM
Tue Jan 5 11:33:07 PST 2010


You are right. We can do it now.

-- John  (on my iPhone)

On Jan 5, 2010, at 10:56 AM, Vladimir Kozlov <Vladimir.Kozlov at Sun.COM>  
wrote:

> Why you not did it?:
>
> // FIXME: Hide this match logic in methodOopDesc or  
> MethodHandleCompiler.
>

> Should you also check for EnableMethodHandles flag in new code?
>
> The next code fix only rbt. What about _rtype == T_ARRAY?
>
> +   if (rbt == T_ARRAY)  rbt = T_OBJECT;
>
> Thanks,
> Vladimir
>
> Christian Thalinger wrote:
>> http://cr.openjdk.java.net/~twisti/6914206/webrev.01/
>> Actually these changes are from John, I'm just doing the webrev and  
>> the
>> push.
>> -- Christian


More information about the hotspot-compiler-dev mailing list