RFR 8199467 Compilation Errors in libinstrument Reentrancy.c with VS2017

Roger Riggs Roger.Riggs at Oracle.com
Wed Mar 21 19:13:52 UTC 2018


Please review a small change to avoid sign extension in libinstrument/ 
Reentrancy.c
to correct an compilation warning with vs2017.

diff --git a/src/java.instrument/share/native/libinstrument/Reentrancy.c 
b/src/java.instrument/share/native/libinstrument/Reentrancy.c
--- a/src/java.instrument/share/native/libinstrument/Reentrancy.c
+++ b/src/java.instrument/share/native/libinstrument/Reentrancy.c
@@ -90,7 +90,7 @@ assertTLSValue( jvmtiEnv *      jvmtienv
                  jthread         thread,
                  const void *    expected) {
      jvmtiError  error;
-    void *      test = (void *) 0x99999999;
+    void *      test = (void *) 0x99999999ul;

      /* now check if we do a fetch we get what we wrote */
      error = (*jvmtienv)->GetThreadLocalStorage(

Issue:
   https://bugs.openjdk.java.net/browse/JDK-8199467

Thanks, Roger



More information about the serviceability-dev mailing list