RFR: Cleanups, TODOs, asserts (part 1) [v2]
Aleksey Shipilev
shade at openjdk.org
Thu Apr 6 17:01:59 UTC 2023
On Thu, 6 Apr 2023 16:55:29 GMT, William Kemper <wkemper at openjdk.org> wrote:
>> Aleksey Shipilev has updated the pull request incrementally with one additional commit since the last revision:
>>
>> New assert fires, but instead of fixing it, remove useless live_memory completely
>
> src/hotspot/share/gc/shenandoah/c2/shenandoahBarrierSetC2.cpp line 900:
>
>> 898: } else {
>> 899: return true;
>> 900: }
>
> This looks off.
Yes, this reverts to "weird" upstream state:
https://builds.shipilev.net/patch-openjdk-shenandoah/src/hotspot/share/gc/shenandoah/c2/shenandoahBarrierSetC2.cpp.sdiff.html
No reason to have whitespace changes if that is the only change in the hunk.
-------------
PR Review Comment: https://git.openjdk.org/shenandoah/pull/245#discussion_r1160047465
More information about the shenandoah-dev
mailing list