RFR: Cleanups, TODOs, asserts (part 1) [v5]

Aleksey Shipilev shade at openjdk.org
Tue Apr 11 07:45:34 UTC 2023


On Mon, 10 Apr 2023 23:17:33 GMT, Y. Srinivas Ramakrishna <ysr at openjdk.org> wrote:

>> Aleksey Shipilev has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 15 commits:
>> 
>>  - Merge branch 'master' into upstream-cleanup-1
>>  - Refine TODO
>>  - Remove unnecessary comment
>>  - Fix a little indentation error
>>  - New assert fires, but instead of fixing it, remove useless live_memory completely
>>  - Reverse bad fix
>>  - More fixes
>>  - More work
>>  - More improvements
>>  - More touchups
>>  - ... and 5 more: https://git.openjdk.org/shenandoah/compare/461f0831...c0af0d7e
>
> src/hotspot/share/gc/shenandoah/c2/shenandoahBarrierSetC2.cpp line 1020:
> 
>> 1018:     Node* addp = shift->unique_out();
>> 1019:     for (DUIterator_Last jmin, j = addp->last_outs(jmin); j >= jmin; --j) {
>> 1020:       Node *mem = addp->last_out(j);
> 
> `Node* mem`

Done!

> src/hotspot/share/gc/shenandoah/shenandoahConcurrentMark.cpp line 181:
> 
>> 179:   ShenandoahConcurrentWorkerSession worker_session(worker_id);
>> 180:   ShenandoahObjToScanQueue* q = _queue_set->queue(worker_id);
>> 181:   ShenandoahObjToScanQueue* old = (_old_queue_set == nullptr) ?
> 
> call this `old_q` too like you did above at line 121?

Done.

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

PR Review Comment: https://git.openjdk.org/shenandoah/pull/245#discussion_r1162421014
PR Review Comment: https://git.openjdk.org/shenandoah/pull/245#discussion_r1162420612


More information about the shenandoah-dev mailing list