RFR: 8252889: C2 crashes when disabling InsertMemBarAfterArraycopy
Aleksey Shipilev
shade at openjdk.java.net
Tue Sep 8 10:23:03 UTC 2020
On Tue, 8 Sep 2020 10:04:35 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:
> The InsertMemBarAfterArraycopy product flag (enabled by default) is broken. I propose to simply remove it.
> The corresponding CSR (JDK-8252890) is still waiting for approval.
>
> Thanks,
> Tobias
Synopsis might be changed to "Obsolete -XX:+InsertMemBarAfterArraycopy".
src/hotspot/share/runtime/arguments.cpp line 556:
> 554: { "UseSemaphoreGCThreadsSynchronization", JDK_Version::undefined(), JDK_Version::jdk(16), JDK_Version::jdk(17) },
> 555: { "ForceNUMA", JDK_Version::jdk(15), JDK_Version::jdk(16), JDK_Version::jdk(17) },
> 556: { "InsertMemBarAfterArraycopy", JDK_Version::undefined(), JDK_Version::jdk(16), JDK_Version::jdk(17) },
This seems like a product flag, so needs to follow `ForceNUMA` example? E.g. 16 -> 17 -> 18? And a provisional CSR?
src/hotspot/share/opto/macroArrayCopy.cpp line 1106:
> 1104: RegionNode* slow_region = new RegionNode(1);
> 1105: transform_later(slow_region);
> 1106:
This cleanup does not seem related?
-------------
Changes requested by shade (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/69
More information about the hotspot-dev
mailing list