RFR: JDK-8273914: Indy string concat changes order of operations [v3]

Liam Miller-Cushon cushon at openjdk.java.net
Fri Jan 7 11:32:16 UTC 2022


On Tue, 12 Oct 2021 07:23:32 GMT, Jan Lahoda <jlahoda at openjdk.org> wrote:

>> Liam Miller-Cushon has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Document the rationale for the jasm changes
>
> How about code like:
> 
>         StringBuilder builder2 = new StringBuilder("foo");
>         Object oo = builder2;
>         oo += "" + builder2.append("bar"); 
> 
> 
> Should that be covered as well? From looking at the patch (not really trying it), it does not seem to be covered?

@lahodaj would you be willing to review for the compiler group?

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

PR: https://git.openjdk.java.net/jdk/pull/5844


More information about the compiler-dev mailing list