Looks fine Kris (given the same change is being made to
hotspot/src/share/vm/prims/jvm.h)
I can sponsor this for you. We can use the same bug number seeing as
these are two separate repositories.
-Chris.
On 07/06/2012 21:10, Krystal Mok wrote:
Hi all,
As discussed on hotspot-compiler-dev [1], I came across a typo in the
exported jvm.h in the jdk workspace while working on 7174218.
The fix is simple [2], and sync up with the changes in 7174218:
diff -r 7def50698e78 src/share/javavm/export/jvm.h
--- a/src/share/javavm/export/jvm.h Thu May 24 16:15:58 2012 -0700
+++ b/src/share/javavm/export/jvm.h Wed Jun 06 01:15:50 2012 +0800
@@ -595,7 +595,7 @@
JVM_AssertionStatusDirectives(JNIEnv *env, jclass unused);
/*
- * java.util.concurrent.AtomicLong
+ * java.util.concurrent.atomic.AtomicLong
*/
JNIEXPORT jboolean JNICALL
JVM_SupportsCX8(void);
Could someone review and sponsor this fix, please?
Regards,
Kris Mok
[1]:
http://mail.openjdk.java.net/pipermail/hotspot-compiler-dev/2012-June/007798.html
[2]:
https://gist.github.com/2876357#file_remove_atomic_long_cs_impl_jdk.patch
