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