RFR: 8306428: RunThese30M.java crashed with assert(early->flag() == current->flag() || early->flag() == mtNone)

Fri May 26 17:38:08 UTC 2023

On Thu, 18 May 2023 22:25:13 GMT, Gerard Ziemski <gziemski at openjdk.org> wrote:

> Based on Thomas' comments from the bug, we fix it by removing the faulty assert, and instead we handle the case of "**same site, different flags**", as `old site` and `new site`.

@tstuefe @dholmes-ora I closed PR 13993, because I thought it would take me a while to think of a way to fix this correctly, but after thinking for a bit I came up with this one, hence new PR. Sorry for making you jump.

If Thomas likes this approach I will move it out of draft.

Thank you Johan and Thomas!

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

PR Comment: https://git.openjdk.org/jdk/pull/14052#issuecomment-1553742780
PR Comment: https://git.openjdk.org/jdk/pull/14052#issuecomment-1553743992
PR Comment: https://git.openjdk.org/jdk/pull/14052#issuecomment-1564709119