RFR: 8244224: Implementation of JEP 381: Remove the Solaris and SPARC Ports (hotspot)

Thomas Schatzl thomas.schatzl at oracle.com
Mon May 4 09:11:56 UTC 2020


Hi,

On 04.05.20 10:28, Stefan Karlsson wrote:
> Hi Mikael,
> 
> On 2020-05-04 07:12, Mikael Vidstedt wrote:
>>
>> Please review this change which implements part of JEP 381:
>>
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8244224
>> webrev: 
>> http://cr.openjdk.java.net/~mikael/webrevs/8244224/webrev.00/hotspot/open/webrev/ 
>>
> 
> I went over this patch and collected some comments:
> 
> src/hotspot/share/adlc/output_c.cpp
> src/hotspot/share/adlc/output_h.cpp
> 
> Awkward code layout after change to.
> 
> 
> src/hotspot/share/c1/c1_Runtime1.cpp
> src/hotspot/share/classfile/classListParser.cpp
> src/hotspot/share/memory/arena.hpp
> src/hotspot/share/opto/chaitin.cpp
> test/hotspot/jtreg/gc/TestCardTablePageCommits.jav >
 > Surrounding comments still refers to Sparc and/or Solaris.
 >
 > There are even more places if you search in the rest of the HotSpot
 > source. Are we leaving those for a separate cleanup pass?

In addition to "sparc", "solaris", also "solstudio"/"Sun Studio"/"SS 
compiler bug"/"niagara" yield some search (=grep) results.

Some of these locations look like additional RFEs.

Thanks,
   Thomas


More information about the hotspot-gc-dev mailing list