RFR: 8319867: GenShen: Make old regions parseable at end of concurrent cycles
William Kemper
wkemper at openjdk.org
Sat Nov 11 00:47:32 UTC 2023
On Sat, 11 Nov 2023 00:11:32 GMT, Y. Srinivas Ramakrishna <ysr at openjdk.org> wrote:
>> assert(_state != BOOTSTRAP && _state != WAITING_FOR_BOOTSTRAP, "Illegal transition");
>
> I am thinking that the most parsimonious way of covering all bases is by associating a pre-condition with entering each state and a post-condition with exiting each state, and verifying those conditions in the transition validation methods. But may be that can be done as a separate PR if it makes sense.
Hmm - I thought that's what this code is doing. Or rather, it is asserting what conditions must hold to _be_ in this state (checking pre/post conditions simultaneously).
-------------
PR Review Comment: https://git.openjdk.org/shenandoah/pull/355#discussion_r1390064292
More information about the shenandoah-dev
mailing list