RFR: 8252103: Parallel heap inspection for ParallelScavengeHeap [v9]
Lin Zang
lzang at openjdk.java.net
Wed Nov 4 10:53:58 UTC 2020
On Wed, 4 Nov 2020 09:56:31 GMT, Thomas Schatzl <tschatzl at openjdk.org> wrote:
>> Dear @tschatzl ,
>> Thanks for your proposed fix, I also prefer it because it could avoid the conversion between signed and unsigned. I will make a new commit to incorporate it.
>>
>> Thanks!
>> Lin
>
> I recommend changing the `(size_t)-1` assignment to `SIZE_MAX`.
>
> While there is (to my knowledge) no compiler (that supports the JVM) that does not use two's complement for integer representation, officially only with C++20 (or later) the result of this operation is exactly defined. We know that other code just does the cast as this does ;), but if you use `SIZE_MAX`, the comment above the definition of `InvalidIndex` can also be removed then.
>
> I would also move `InvalidIndex` as the first in the list of constants because it's not really related to numbers assigned because of the spaces (while `NumNonOldGenClaims` is).
>
> Lgtm otherwise. Thanks for bearing with us.
Dear @tschatzl @kstefanj and @albertnetymk,
So appericated for your suggestion and guidance. I have upload a new commit.
would you like to help review it again.
BRs,
Lin
-------------
PR: https://git.openjdk.java.net/jdk/pull/25
More information about the hotspot-gc-dev
mailing list