RFR: 8347405: MergeStores with reverse bytes order value [v4]
kuaiwei
duke at openjdk.org
Fri Jan 24 12:51:27 UTC 2025
On Fri, 24 Jan 2025 02:03:13 GMT, kuaiwei <duke at openjdk.org> wrote:
>> @kuaiwei Feel free to ping me again here whenever this is ready for a re-review :)
>
> @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 this is a small test-fix for big-endian [reinrich at 188a97a](https://github.com/reinrich/jdk/commit/188a97a0deeb6bb912162846d02901079968c39f) I havn't seen any test failures in broader testing. Will repeat for latest change.
@reinrich ,Thanks for testing. The patch is added.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/23030#issuecomment-2612454188
More information about the hotspot-compiler-dev
mailing list