RFR: Cleanups, TODOs, asserts (part 2)

Aleksey Shipilev shade at openjdk.org
Wed Apr 12 08:07:27 UTC 2023


On Tue, 11 Apr 2023 17:25:22 GMT, Y. Srinivas Ramakrishna <ysr at openjdk.org> wrote:

>> I have been reading the openjdk/shenandoah vs openjdk/jdk diff and these are the simple code massages we can do to improve and bullet-proof the code, hopefully without changing the semantics. This would continue with more parts later.
>> 
>> Additional testing:
>>  - [ ] macos-aarch64-server-fastdebug (some pre-existing failures)
>>  - [ ] GHA builds
>
> src/hotspot/share/gc/shenandoah/shenandoahMarkingContext.hpp line 63:
> 
>> 61: 
>> 62:   // Simple versions of marking accessors, to be used outside of marking (e.g. no possible concurrent updates)
>> 63:   // TODO: Do these really need to be const?
> 
> It's generally considered good hygiene to `const` read accessors. Any reason you would _not_ want them `const`'d?

I'd like to keep the difference vs upstream at minimum. So my question is, do we need these additions for some build/performance reasons, or this is just a hygienic measure? The hygienic fixes in existing upstream code can be done separately straight in upstream.

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

PR Review Comment: https://git.openjdk.org/shenandoah/pull/252#discussion_r1163772355


More information about the shenandoah-dev mailing list