RFR: 8351468: C2: array fill optimization assigns wrong type to intrinsic call [v3]

Fri Mar 21 14:59:09 UTC 2025

On Thu, 20 Mar 2025 15:16:16 GMT, Tobias Hartmann <thartmann at openjdk.org> wrote:

>> Roberto Castañeda Lozano has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Explicitly disable optimization for mismatching stores; add positive and negative tests
>
> Good catch and nice tests! The fix looks good to me.

@TobiHartmann @merykitty @eme64 Thanks for reviewing! I will update the tests as suggested by @eme64 and re-run testing over the weekend.
@RealFYang I enabled the new IR tests in `test/hotspot/jtreg/compiler/loopopts/TestArrayFillIntrinsic.java` on riscv64 because this platform seems to handle array fill intrinsification similarly to x64 and aarch64. Would you like to test it before integration?

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

PR Comment: https://git.openjdk.org/jdk/pull/24005#issuecomment-2743614014