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