RFR: 8251451: Shenandoah: Remark ObjectSynchronizer roots with I-U

Aleksey Shipilev shade at redhat.com
Wed Aug 12 10:42:09 UTC 2020


On 8/12/20 12:36 PM, Roman Kennke wrote:
> http://cr.openjdk.java.net/~rkennke/JDK-8251451/webrev.01/

Yes, that's better. The indenting is off, though: lots of space before "_claimed", and "public" is
moved. Should be:

private:
  ShenandoahConcurrentMark* _cm;
  TaskTerminator*           _terminator;
  bool                      _dedup_string;
  ShenandoahSharedFlag      _claimed_syncroots;

public:
  ShenandoahFinalMarkingTask(ShenandoahConcurrentMark* cm, TaskTerminator* terminator, bool
dedup_string) :

No need for another review.

-- 
Thanks,
-Aleksey



More information about the shenandoah-dev mailing list