United StatesChange Country, Oracle Worldwide Web Sites Communities I am a... I want to...
Bug ID: JDK-6974017 Upgrade required Solaris Studio compilers to 5.10 (12 update 1 + patches)
JDK-6974017 : Upgrade required Solaris Studio compilers to 5.10 (12 update 1 + patches)

Details
Type:
Enhancement
Submit Date:
2010-08-03
Status:
Resolved
Updated Date:
2014-03-03
Project Name:
JDK
Resolved Date:
2010-09-16
Component:
infrastructure
OS:
generic
Sub-Component:
build
CPU:
generic
Priority:
P3
Resolution:
Fixed
Affected Versions:
7
Fixed Versions:

Related Reports
Backport:

Sub Tasks

Description
Essentially, remove this warning when using 12 update 1 + patches compiler:

WARNING: The solaris compiler is not version SS12 5.9 
       Specifically the Sun Studio 12 compiler. 
        You appear to be using SS13: 5.10 
       The compiler was obtained from the following location: 
           /java/devtools/i386/SUNWspro/SS12u1/bin/ 

It should now warn when using an older version. Also, SS13 comment is wrong, should be SS12u1.

Similar change will be needed in hotspot.

diff --git a/make/common/shared/Compiler-sun.gmk b/make/common/shared/Compiler-sun.gmk
--- a/make/common/shared/Compiler-sun.gmk
+++ b/make/common/shared/Compiler-sun.gmk
@@ -59,8 +59,9 @@
 endif
 
 # Get compiler version
-_CC_VER :=$(shell $(CC) -V 2>&1 | $(HEAD) -n 1)
-CC_VER  :=$(call GetVersion,"$(_CC_VER)")
+_CC_VER      :=$(shell $(CC) -V 2>&1 | $(HEAD) -n 1)
+CC_VER       :=$(call GetVersion,"$(_CC_VER)")
+CC_MINORVER  :=$(call MinorVersion,$(CC_VER))
 
 # Name of compilers being used
 COMPILER_VERSION-5.7  = SS10
@@ -69,8 +70,8 @@
 COMPILER_NAME-5.8     = Sun Studio 11
 COMPILER_VERSION-5.9  = SS12
 COMPILER_NAME-5.9     = Sun Studio 12
-COMPILER_VERSION-5.10 = SS13
-COMPILER_NAME-5.10    = Sun Studio 13
+COMPILER_VERSION-5.10 = SS12u1
+COMPILER_NAME-5.10    = Sun Studio 12u1
 COMPILER_VERSION      = $(COMPILER_VERSION-$(CC_VER))
 COMPILER_NAME         = $(COMPILER_NAME-$(CC_VER))
 
@@ -112,8 +113,8 @@
   XARCH_OPTION_OLD/64      += -xarch=amd64
   LINT_XARCH_OPTION_OLD/64 += -Xarch=amd64
 endif
-# Pick the options we want based on the compiler being used.
-ifeq ($(shell expr $(CC_VER) \>= 5.9), 1)
+# Pick the options we want based on the compiler being used. (5.9 or newer)
+ifeq ($(shell expr $(CC_MINORVER) \>= 9), 1)
   XARCH_OPTION/32 = $(XARCH_OPTION_NEW/32)
   XARCH_OPTION/64 = $(XARCH_OPTION_NEW/64)
   LINT_XARCH_OPTION/32 = $(LINT_XARCH_OPTION_NEW/32)




diff --git a/make/common/shared/Defs-versions.gmk b/make/common/shared/Defs-versions.gmk
--- a/make/common/shared/Defs-versions.gmk
+++ b/make/common/shared/Defs-versions.gmk
@@ -123,7 +123,7 @@
   REQUIRED_COMPILER_NAME      = Sun Studio 12
   REQUIRED_COMPILER_VERSION   = SS12
   ifeq ($(CC_VERSION),sun)
-    REQUIRED_CC_VER           = 5.9
+    REQUIRED_CC_VER           = 5.10
   endif
   ifeq ($(CC_VERSION),gcc)
     REQUIRED_CC_VER           = 3.4.3
@@ -145,7 +145,7 @@
     REQUIRED_CC_VER           = 4.3.0
   endif
   ifeq ($(CC_VERSION),sun)
-    REQUIRED_CC_VER           = 5.9
+    REQUIRED_CC_VER           = 5.10
   endif
 endif

                                    

Comments
EVALUATION

As soon as JPRT is setup.
                                     
2010-08-03
EVALUATION

http://hg.openjdk.java.net/jdk7/build/jdk/rev/c5f6cd3bd70b
http://hg.openjdk.java.net/jdk7/build/rev/782c0c738f6d
                                     
2010-09-08



Hardware and Software, Engineered to Work Together