RFR: 8243325: Cleanup TaskQueueSuper<>::peek
Thomas Schatzl
thomas.schatzl at oracle.com
Tue May 5 09:03:23 UTC 2020
Hi,
On 05.05.20 07:39, Kim Barrett wrote:
> Please review this change to some assertions in taskTerminator that
> are verifying all the queues are empty. Previously this was
> accomplished by using the peek operation on the queues, verifying all
> returned false. That's the only use of the peek operation. This
> change replaces that usage with new assert_empty helper functions, and
> eliminates the now unused peek operations.
>
> CR:
> https://bugs.openjdk.java.net/browse/JDK-8243325
>
> Webrev:
> https://cr.openjdk.java.net/~kbarrett/8243325/open.00/
>
> Testing:
> mach5 tier1-5, in combination with fix for JDK-8244307.
> Locally (linux-x64) hotspot:tier1 without JDK-8244307.
>
looks good.
Thomas
More information about the hotspot-gc-dev
mailing list