RFR: 8290705: StringConcat::validate_mem_flow asserts with "unexpected user: StoreI" [v2]

[Xin Liu]

On Wed, 27 Jul 2022 00:49:52 GMT, Xin Liu <xliu at openjdk.org> wrote:

>> Tobias Hartmann has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Modified debug printing code
>
> test/hotspot/jtreg/compiler/stringopts/SideEffectBeforeConstructor.jasm line 54:
> 
>> 52: 		putstatic	Field result:"I";
>> 53: 		aload_0;
>> 54: 		invokespecial	Method java/lang/StringBuffer."<init>":"(Ljava/lang/String;)V";
> 
> hi, @TobiHartmann , 
> Is here the reason why you said "javac would not generate such code"?
> I don't think javac will insert "SideEffectBeforeConstructor::result++" btween new and invokespecial.

I tried that. I don't think there's a way to generate code like that using javac. 
So we fix this bug because somebody may emit weird bytecode sequences using asm?

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

PR: https://git.openjdk.org/jdk/pull/9589