RFR (S): 8200255: Remove G1CMTask::_concurrent
sangheon.kim
sangheon.kim at oracle.com
Wed Mar 28 17:45:38 UTC 2018
Hi Thomas,
On 03/26/2018 11:25 AM, Thomas Schatzl wrote:
> Hi all,
>
> can I have reviews for this small change that removes
> G1CMTask::_concurrent. That member is a duplicate of
> G1ConcurrentMark::_concurrent, but there does not seem to be any
> advantage of keeping this duplicate, as it's only used in asserts or
> non-performance critical code.
>
> This removes some imho unnecessary code, so I suggest to do that.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8200255
> Webrev:
> http://cr.openjdk.java.net/~tschatzl/8200255/webrev/
> Testing:
> local compilation, gcbasher, testing with other changes
Looks good to me too.
Thanks,
Sangheon
>
> Thanks,
> Thomas
More information about the hotspot-gc-dev
mailing list