RFR(S): 8173335: Improve logging for j.l.ref.reference processing
Aleksey Shipilev
shade at redhat.com
Mon Aug 7 11:53:21 UTC 2017
On 08/04/2017 08:46 AM, sangheon wrote:
> Webrev:
> http://cr.openjdk.java.net/~sangheki/8173335/webrev.4 (full)
> http://cr.openjdk.java.net/~sangheki/8173335/webrev.4_to_3b (inc)
Looks good to me. It takes quite a bit of G1 in its sweep, but I guess G1 folks are fine with that?
Minor nit:
*) In referenceProcessor.cpp, these line should be separated by a newline, because indenting is
confusing:
195 ReferenceProcessorPhaseTimes* phase_times) {
196 double start_time = os::elapsedTime();
Thanks,
-Aleksey
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: OpenPGP digital signature
URL: <https://mail.openjdk.org/pipermail/hotspot-gc-dev/attachments/20170807/761150eb/signature.asc>
More information about the hotspot-gc-dev
mailing list