RFR: 8253420: Refactor HeapRegionManager::find_highest_free [v2]
Albert Mingkun Yang
ayang at openjdk.java.net
Tue Jan 26 09:12:53 UTC 2021
> Using for-loop to make the number of iterations more explicit. Direct backward iteration, `for (uint curr = reserved_length() - 1; curr >= 0; curr--)` doesn't work due to underflow of `uint` type. Therefore, I went for current approach.
>
> Test: hotspot_gc
Albert Mingkun Yang has updated the pull request incrementally with one additional commit since the last revision:
review
-------------
Changes:
- all: https://git.openjdk.java.net/jdk/pull/2193/files
- new: https://git.openjdk.java.net/jdk/pull/2193/files/c734b63a..67ce558f
Webrevs:
- full: https://webrevs.openjdk.java.net/?repo=jdk&pr=2193&range=01
- incr: https://webrevs.openjdk.java.net/?repo=jdk&pr=2193&range=00-01
Stats: 2 lines in 1 file changed: 0 ins; 1 del; 1 mod
Patch: https://git.openjdk.java.net/jdk/pull/2193.diff
Fetch: git fetch https://git.openjdk.java.net/jdk pull/2193/head:pull/2193
PR: https://git.openjdk.java.net/jdk/pull/2193
More information about the hotspot-gc-dev
mailing list