Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-6234975

Hotspot Solaris SS10 build needs cleanup

XMLWordPrintable

    • Icon: Enhancement Enhancement
    • Resolution: Duplicate
    • Icon: P4 P4
    • 7
    • 6
    • hotspot
    • generic
    • solaris

      There are some areas in Solaris sparc/x86 makefiles that may lower optimization levels or specify other workarounds for limitations of the cc55 compiler. Once we upgrade to the cc57 (SS10) compiler these assumptions may no longer be valid and we may want to remove the workarounds, potentially improving performance.

      Two locations that I know about are:
      1. sparcWorks.make:114 sets x86 opt level to -xO3 to workaround a cc55 bug; we might be able to up this to -xO4 for cc57.

      2. "optimized" build should be built & tested to ensure that the cc57 workarounds of 6234407/6233962 mentioned in fastdebug.make are not needed in optimized.make. Thanks to ###@###.### for pointing this out.

      There are surely other areas that need cleanup after the SS10 switch - needs investigation.


      ###@###.### 2005-03-02 16:06:51 GMT

      Also there are some new compilation warnings with SS10 in fastdebug:
      "/nightly/ws/src/share/vm/oops/generateOopMap.hpp", line 118: Warning: The variable s has not yet been assigned a value.
      "/nightly/ws/src/share/vm/oops/generateOopMap.hpp", line 118: Warning: The variable s has not yet been assigned a value.

      "/nightly/ws/src/share/vm/code/vtableStubs.cpp", line 122: Warning: Initializing const VtableStub& to a NULL value.

      Or should this be a different bug?
      ###@###.### 2005-03-21 19:19:49 GMT
      As a result of the fix for 6295979 the cc57 workarounds of 6234407/6233962 mentioned in fastdebug.make will be gone as of b48. The C++ compiler bugs have been fixed so there should be no longer be any need to check this particular issue with respect to the optimized build.

            duke J. Duke
            sbohne Steve Bohne (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved:
              Imported:
              Indexed: