Thanks for the review, Karen.

On 1/18/2013 10:15 AM, Karen Kinnear wrote:
I like the additional checking. I wonder if you could possibly modify this to 
rename
the API to JVM_IsVMGeneratedMethodIx - since that might be clearer that this is 
has internal
vm meaning which is not related to any of the specifications on the constant 
pool etc.

I have made the suggested changes.

Please review the updated changes at 
http://cr.openjdk.java.net/~bharadwaj/8004967/alt_impl

Hotspot tree changes : 
http://cr.openjdk.java.net/~bharadwaj/8004967/alt_impl/hotspot/webrev/
JDK tree changes : 
http://cr.openjdk.java.net/~bharadwaj/8004967/alt_impl/jdk/webrev/

Thanks,

Bharadwaj

Reply via email to