RFR: 8209347: SATBMarkQueue.cpp should not need jvm.h

Thomas Schatzl thomas.schatzl at oracle.com
Mon Aug 13 10:10:58 UTC 2018


Hi,

On Sun, 2018-08-12 at 22:14 -0400, Kim Barrett wrote:
> Please review this small cleanup, replacing a couple calls to
> jio_snprintf with calls to os::snprintf, and updating includes
> accordingly.
> 
> This changeset is based on the in-review changes for JDK-8209396,
> JDK-8209345, and JDK-8209346.
> http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2018-August/022
> 868.html
> http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2018-August/022
> 869.html
> http://mail.openjdk.java.net/pipermail/hotspot-gc-dev/2018-August/022
> 870.html
> 
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8209347
> 
> Webrev:
> http://cr.openjdk.java.net/~kbarrett/8209347/open.00/
> 
> Testing:
> Mach5 tier1-3, hs-tier4-5, in conjunction with JDK-8209345 and JDK-
> 8209346.
> 

  looks good.

Thomas



More information about the hotspot-gc-dev mailing list