RFR: 8261842: Shenandoah: cleanup ShenandoahHeapRegionSet

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR: 8261842: Shenandoah: cleanup ShenandoahHeapRegionSet

Aleksey Shipilev-5
There are a couple of stale/unused methods in ShenandoahHeapRegionSet that we can eliminate instead of improving them, for example in JDK-8261838.

Additional testing:
 - [x] Linux x86_64 `hotspot_gc_shenandoah`

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

Commit messages:
 - Update
 - 8261842: Shenandoah: cleanup ShenandoahHeapRegionSet

Changes: https://git.openjdk.java.net/jdk/pull/2592/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jdk&pr=2592&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8261842
  Stats: 88 lines in 3 files changed: 0 ins; 82 del; 6 mod
  Patch: https://git.openjdk.java.net/jdk/pull/2592.diff
  Fetch: git fetch https://git.openjdk.java.net/jdk pull/2592/head:pull/2592

PR: https://git.openjdk.java.net/jdk/pull/2592
Reply | Threaded
Open this post in threaded view
|

Re: RFR: 8261842: Shenandoah: cleanup ShenandoahHeapRegionSet

Roman Kennke
On Tue, 16 Feb 2021 19:11:27 GMT, Aleksey Shipilev <[hidden email]> wrote:

> There are a couple of stale/unused methods in ShenandoahHeapRegionSet that we can eliminate instead of improving them, for example in JDK-8261838.
>
> Additional testing:
>  - [x] Linux x86_64 `hotspot_gc_shenandoah`

Ok!

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

Marked as reviewed by rkennke (Reviewer).

PR: https://git.openjdk.java.net/jdk/pull/2592
Reply | Threaded
Open this post in threaded view
|

Integrated: 8261842: Shenandoah: cleanup ShenandoahHeapRegionSet

Aleksey Shipilev-5
In reply to this post by Aleksey Shipilev-5
On Tue, 16 Feb 2021 19:11:27 GMT, Aleksey Shipilev <[hidden email]> wrote:

> There are a couple of stale/unused methods in ShenandoahHeapRegionSet that we can eliminate instead of improving them, for example in JDK-8261838.
>
> Additional testing:
>  - [x] Linux x86_64 `hotspot_gc_shenandoah`

This pull request has now been integrated.

Changeset: d1950335
Author:    Aleksey Shipilev <[hidden email]>
URL:       https://git.openjdk.java.net/jdk/commit/d1950335
Stats:     88 lines in 3 files changed: 0 ins; 82 del; 6 mod

8261842: Shenandoah: cleanup ShenandoahHeapRegionSet

Reviewed-by: rkennke

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

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