On Tue, 9 May 2023 01:23:23 GMT, David Holmes <dhol...@openjdk.org> wrote:
>> Paul Hohensee has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Implement 32-bit linux Atomic::add() > > Also note that the current placement of the `incr_exited_allocated_bytes` > skips accounting for hidden threads because those ThreadService methods skip > hidden threads. You need to do this update as one of the last actions of a > terminating/detaching thread. Looks good now except for the points already mentioned by @dholmes-ora . ------------- PR Comment: https://git.openjdk.org/jdk/pull/13814#issuecomment-1547855460
