RFR: 8264788: Make SequentialSubTasksDone use-once [v2]

Thomas Schatzl tschatzl at openjdk.java.net
Thu Apr 8 15:05:42 UTC 2021


On Thu, 8 Apr 2021 09:02:20 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:

>> Thomas Schatzl has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   ayang review
>
> Looking at how `SequentialSubTasksDone` is used, I don't really see the benefit of having `assert(_num_claimed == _num_tasks)` in the destructor. If we drop that assertion, `Atomic::add` is more readable.
> 
> I prefer `Atomic::add` with the assertion removed, but this version is fine as well.

I changed to the `Atomic::add` but kept the assert (modified).

-------------

PR: https://git.openjdk.java.net/jdk/pull/3372



More information about the hotspot-gc-dev mailing list