RFR: 8359603: Missed optimization in PhaseIterGVN for redundant ConvX2Y->ConvY2X->ConvX2Y sequences due to missing notification in PhaseIterGVN::add_users_of_use_to_worklist [v3]

[Benoît Maillard]

On Wed, 23 Jul 2025 08:44:36 GMT, Christian Hagedorn <chagedorn at openjdk.org> wrote:

>> Benoît Maillard has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8359603: Reduce number of iterations in tests
>
> test/hotspot/jtreg/compiler/c2/TestEliminateRedundantConversionSequences.java line 94:
> 
>> 92: 
>> 93:     public static void main(String[] strArr) {
>> 94:         for (int i = 0; i < 50_000; ++i) {
> 
> Do you really need 50000 iterations each? Would less also trigger the bug?

I was able to reduce it to ~1550

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26368#discussion_r2236079880