RFR: 8337066: Repeated call of StringBuffer.reverse with double byte string returns wrong result

Vladimir Kozlov kvn at openjdk.org
Fri Sep 27 19:17:35 UTC 2024


On Fri, 27 Sep 2024 19:12:02 GMT, Igor Veresov <iveresov at openjdk.org> wrote:

>> Normally loads would not have a MergeMem as their memory input at this stage. `Ideal()` splits the memory, so that the memory input of a load is as precise as possible. So I think, that loop is benign for a regular load.
>
> Also if there is a MergeMem as a root for some weird reason then `insert_anti_dependencies()` may very well miss an interfering store. So we'd have to do this loop for correctness.

Okay

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21222#discussion_r1779051056


More information about the hotspot-compiler-dev mailing list