RFR: 8347405: MergeStores with reverse bytes order value [v4]

[Emanuel Peter]

On Thu, 30 Jan 2025 13:09:39 GMT, kuaiwei <duke at openjdk.org> wrote:

>>> > @eme64 , I made new change to merge more stores. It could merge pattern like
>>> > ```c++
>>> > a[0] = vL <<  8;
>>> > a[1] = vL << 16;
>>> > a[2] = vL << 24;
>>> > a[3] = vL << 32;
>>> > ```
>>> > 
>>> > 
>>> >     
>>> >       
>>> >     
>>> > 
>>> >       
>>> >     
>>> > 
>>> >     
>>> >   
>>> > It passed tier1 tests in x86 and aarch64 platform. May I ask you to review it again? Thanks.
>>> 
>>> @kuaiwei That is a nice idea / generalization!
>>> 
>>> However: I think we should really do that in a separate RFE. Because it needs to be thoroughly tested, and that would blow up the size of your PR, and make reviewing much slower. The reverse order is already complex enough for us to review ;)
>> 
>> Ok, I reverted the change.
>
>> @kuaiwei Thanks for agreeing to do to separate out the additional improvement! 😊
>> 
>> Is this ready for a next round of reviews?
> 
> @eme64 , yes, I think the patch is ready for review. Could you take time to check it? Thanks.

@kuaiwei Sounds good. Thanks for all the work you put in. These things tend to come out more complicated than one first thinks 😅 

I won't have time today, but I ran testing for commit 16 / v10. Feel free to ping me after the weekend, then I can have a look at the tests, and either tell you about the failures or review the code ;)

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

PR Comment: https://git.openjdk.org/jdk/pull/23030#issuecomment-2626423558