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
