RFR - 8072450: 9-dev build failed on elinux-i586 and rlinux-i586

[Christian Thalinger]

> On Feb 4, 2015, at 3:31 AM, Daniel Fuchs <daniel.fuchs at oracle.com> wrote:
> 
> Hi,
> 
> Please find below a fix for:
> 
> 8072450: 9-dev build failed on elinux-i586 and rlinux-i586
> 
> My fix for JDK-8068730 which was integrated from hs-rt into jdk9-dev
> yesterday is causing a build failure in the jdk9/dev nightly on two platforms.
> It appears that these platforms have a slightly older version
> of the compiler - which chokes on jvm.cpp with the following error:
> 
> hotspot/src/share/vm/prims/jvm.cpp:307: error: integer constant is too large for ���long��� type
> 
> Adding a LL suffix should solve the issue:
> 
> ##############
> 
> diff --git a/src/share/vm/prims/jvm.cpp b/src/share/vm/prims/jvm.cpp
> --- a/src/share/vm/prims/jvm.cpp
> +++ b/src/share/vm/prims/jvm.cpp
> @@ -304,7 +304,7 @@
> // java.lang.System, but we choose to keep it here so that it stays next
> // to JVM_CurrentTimeMillis and JVM_NanoTime
> 
> -const jlong MAX_DIFF_SECS = 0x0100000000;   //  2^32
> +const jlong MAX_DIFF_SECS = 0x0100000000LL; //  2^32

Don’t use LL directly; we have a compiler-dependent macro for this:

CONST64

> const jlong MIN_DIFF_SECS = -MAX_DIFF_SECS; // -2^32
> 
> JVM_LEAF(jlong, JVM_GetNanoTimeAdjustment(JNIEnv *env, jclass ignored, jlong offset_secs))
> 
> ##############
> 
> ( or if you prefer here is a webrev:
>  http://cr.openjdk.java.net/~dfuchs/webrev_8072450/webrev.00 )
> 
> best regards,
> 
> -- daniel
>