RFR (S): 8200255: Remove G1CMTask::_concurrent (original) (raw)
Stefan Johansson stefan.johansson at oracle.com
Wed Mar 28 12:24:21 UTC 2018
- Previous message (by thread): RFR (S): 8200255: Remove G1CMTask::_concurrent
- Next message (by thread): RFR (S): 8200255: Remove G1CMTask::_concurrent
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Looks good, StefanJ
On 2018-03-26 20:25, 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 Thanks, Thomas
- Previous message (by thread): RFR (S): 8200255: Remove G1CMTask::_concurrent
- Next message (by thread): RFR (S): 8200255: Remove G1CMTask::_concurrent
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]