8072482: Cleanup: In jvm.cpp and other shared files declaration of 64bits constants should use the CONST64/UCONST64 macros instead of the LL suffix
authordfuchs
Wed, 11 Feb 2015 17:41:55 +0100
changeset 28956 998d9d0b33a4
parent 28955 c90fb9fd47bd
child 28957 75ca61f0b7c7
child 29068 683bef04fb20
8072482: Cleanup: In jvm.cpp and other shared files declaration of 64bits constants should use the CONST64/UCONST64 macros instead of the LL suffix Reviewed-by: dholmes, coleenp
hotspot/src/share/vm/opto/divnode.cpp
hotspot/src/share/vm/prims/jvm.cpp
--- a/hotspot/src/share/vm/opto/divnode.cpp	Tue Feb 10 22:22:36 2015 +0000
+++ b/hotspot/src/share/vm/opto/divnode.cpp	Wed Feb 11 17:41:55 2015 +0100
@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 1997, 2012, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 1997, 2015, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -219,7 +219,7 @@
 static bool magic_long_divide_constants(jlong d, jlong &M, jint &s) {
   int64_t p;
   uint64_t ad, anc, delta, q1, r1, q2, r2, t;
-  const uint64_t two63 = 0x8000000000000000LL;     // 2**63.
+  const uint64_t two63 = UCONST64(0x8000000000000000);     // 2**63.
 
   ad = ABS(d);
   if (d == 0 || d == 1) return false;
--- a/hotspot/src/share/vm/prims/jvm.cpp	Tue Feb 10 22:22:36 2015 +0000
+++ b/hotspot/src/share/vm/prims/jvm.cpp	Wed Feb 11 17:41:55 2015 +0100
@@ -303,7 +303,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 = 0x0100000000LL; //  2^32
+const jlong MAX_DIFF_SECS = CONST64(0x0100000000); //  2^32
 const jlong MIN_DIFF_SECS = -MAX_DIFF_SECS; // -2^32
 
 JVM_LEAF(jlong, JVM_GetNanoTimeAdjustment(JNIEnv *env, jclass ignored, jlong offset_secs))