RFR: JDK-8312617: SIGSEGV in ConnectionGraph::verify_ram_nodes [v2]

Thu Jul 27 18:23:31 UTC 2023

On Wed, 26 Jul 2023 23:46:54 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

>> Cesar Soares Lucas has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Refactoring use of flags.
>
> Use `_compile->do_reduce_allocation_merges()` instead of `ReduceAllocationMerges` in all places in `escape.cpp`.   You removed one in `adjust_scalar_replaceable_state()`. I see an other at line 51 (`_nodes` field allocation) and at line 401 (where you don't need it - `reducible_merges.size() > 0` check is enough)

Thanks for reviewing @vnkozlov

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

PR Comment: https://git.openjdk.org/jdk/pull/15048#issuecomment-1654166085