Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v8]

2025-05-27 Thread Andrew Haley
On Thu, 15 May 2025 16:03:44 GMT, Andrew Haley wrote: >> This intrinsic is generally faster than the current implementation for >> Panama segment operations for all writes larger than about 8 bytes in size, >> increasing to more than 2* the performance on larger memory blocks on >> Graviton 2,

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v8]

2025-05-22 Thread John R Rose
On Thu, 15 May 2025 16:03:44 GMT, Andrew Haley wrote: >> This intrinsic is generally faster than the current implementation for >> Panama segment operations for all writes larger than about 8 bytes in size, >> increasing to more than 2* the performance on larger memory blocks on >> Graviton 2,

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v8]

2025-05-15 Thread Andrew Dinn
On Thu, 15 May 2025 16:03:44 GMT, Andrew Haley wrote: >> This intrinsic is generally faster than the current implementation for >> Panama segment operations for all writes larger than about 8 bytes in size, >> increasing to more than 2* the performance on larger memory blocks on >> Graviton 2,

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v8]

2025-05-15 Thread Andrew Haley
> This intrinsic is generally faster than the current implementation for Panama > segment operations for all writes larger than about 8 bytes in size, > increasing to more than 2* the performance on larger memory blocks on > Graviton 2, between "panama" (C2 generated, what we use now) and "unsaf

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v6]

2025-05-15 Thread Andrew Haley
On Wed, 14 May 2025 15:36:32 GMT, Andrew Dinn wrote: > Looks good to me. Sorry, please approve again. - PR Comment: https://git.openjdk.org/jdk/pull/25147#issuecomment-2884310593

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v7]

2025-05-15 Thread Andrew Haley
> This intrinsic is generally faster than the current implementation for Panama > segment operations for all writes larger than about 8 bytes in size, > increasing to more than 2* the performance on larger memory blocks on > Graviton 2, between "panama" (C2 generated, what we use now) and "unsaf

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v6]

2025-05-14 Thread Andrew Dinn
On Wed, 14 May 2025 10:47:40 GMT, Andrew Haley wrote: >> This intrinsic is generally faster than the current implementation for >> Panama segment operations for all writes larger than about 8 bytes in size, >> increasing to more than 2* the performance on larger memory blocks on >> Graviton 2,

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v6]

2025-05-14 Thread Per Minborg
On Wed, 14 May 2025 10:47:40 GMT, Andrew Haley wrote: >> This intrinsic is generally faster than the current implementation for >> Panama segment operations for all writes larger than about 8 bytes in size, >> increasing to more than 2* the performance on larger memory blocks on >> Graviton 2,

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v6]

2025-05-14 Thread Andrew Haley
> This intrinsic is generally faster than the current implementation for Panama > segment operations for all writes larger than about 8 bytes in size, > increasing to more than 2* the performance on larger memory blocks on > Graviton 2, between "panama" (C2 generated, what we use now) and "unsaf

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v2]

2025-05-13 Thread Andrew Haley
On Mon, 12 May 2025 09:39:48 GMT, Andrew Haley wrote: > > Looking at the improvements made, I suggest we also change (in > > `SegmentBulkOperations`): > > ``` > > private static final int NATIVE_THRESHOLD_FILL = powerOfPropertyOr("fill", > > Architecture.isAARCH64() ? 18 : 5); > > ``` > > to >

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v5]

2025-05-13 Thread Andrew Haley
> This intrinsic is generally faster than the current implementation for Panama > segment operations for all writes larger than about 8 bytes in size, > increasing to more than 2* the performance on larger memory blocks on > Graviton 2, between "panama" (C2 generated, what we use now) and "unsaf

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v4]

2025-05-13 Thread Andrew Haley
> This intrinsic is generally faster than the current implementation for Panama > segment operations for all writes larger than about 8 bytes in size, > increasing to more than 2* the performance on larger memory blocks on > Graviton 2, between "panama" (C2 generated, what we use now) and "unsaf

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v3]

2025-05-12 Thread Andrew Haley
On Mon, 12 May 2025 09:48:03 GMT, Andrew Dinn wrote: >> Andrew Haley has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Stub stack frame > > src/hotspot/cpu/aarch64/stubGenerator_aarch64.cpp line 2619: > >> 2617: >> 2618: __ bind(tail

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v3]

2025-05-12 Thread Andrew Dinn
On Mon, 12 May 2025 09:37:24 GMT, Andrew Haley wrote: >> This intrinsic is generally faster than the current implementation for >> Panama segment operations for all writes larger than about 8 bytes in size, >> increasing to more than 2* the performance on larger memory blocks on >> Graviton 2,

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v2]

2025-05-12 Thread Andrew Haley
On Mon, 12 May 2025 06:52:12 GMT, Per Minborg wrote: > Looking at the improvements made, I suggest we also change (in > `SegmentBulkOperations`): > > ``` > private static final int NATIVE_THRESHOLD_FILL = powerOfPropertyOr("fill", > Architecture.isAARCH64() ? 18 : 5); > ``` > > to > > ``` >

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v3]

2025-05-12 Thread Andrew Haley
> This intrinsic is generally faster than the current implementation for Panama > segment operations for all writes larger than about 8 bytes in size, > increasing to more than 2* the performance on larger memory blocks on > Graviton 2, between "panama" (C2 generated, what we use now) and "unsaf

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v2]

2025-05-12 Thread Andrew Haley
On Mon, 12 May 2025 09:18:13 GMT, Andrew Dinn wrote: >> Andrew Haley has updated the pull request incrementally with one additional >> commit since the last revision: >> >> generate_unsafecopy_common_error_exit > > src/hotspot/cpu/aarch64/stubGenerator_aarch64.cpp line 2611: > >> 2609: >> 2

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v2]

2025-05-12 Thread Andrew Dinn
On Fri, 9 May 2025 15:39:35 GMT, Andrew Haley wrote: >> This intrinsic is generally faster than the current implementation for >> Panama segment operations for all writes larger than about 8 bytes in size, >> increasing to more than 2* the performance on larger memory blocks on >> Graviton 2,

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v2]

2025-05-11 Thread Per Minborg
On Fri, 9 May 2025 15:39:35 GMT, Andrew Haley wrote: >> This intrinsic is generally faster than the current implementation for >> Panama segment operations for all writes larger than about 8 bytes in size, >> increasing to more than 2* the performance on larger memory blocks on >> Graviton 2,

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v2]

2025-05-11 Thread SendaoYan
On Fri, 9 May 2025 15:39:35 GMT, Andrew Haley wrote: >> This intrinsic is generally faster than the current implementation for >> Panama segment operations for all writes larger than about 8 bytes in size, >> increasing to more than 2* the performance on larger memory blocks on >> Graviton 2,

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory [v2]

2025-05-09 Thread Andrew Haley
> This intrinsic is generally faster than the current implementation for Panama > segment operations for all writes larger than about 8 bytes in size, > increasing to more than 2* the performance on larger memory blocks on > Graviton 2, between "panama" (C2 generated, what we use now) and "unsaf

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory

2025-05-09 Thread Andrew Haley
On Fri, 9 May 2025 14:11:27 GMT, Andrew Haley wrote: > This intrinsic is generally faster than the current implementation for Panama > segment operations for all writes larger than about 8 bytes in size, > increasing to more than 2* the performance on larger memory blocks on > Graviton 2, betw

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory

2025-05-09 Thread Andrew Haley
On Fri, 9 May 2025 14:11:27 GMT, Andrew Haley wrote: > This intrinsic is generally faster than the current implementation for Panama > segment operations for all writes larger than about 8 bytes in size, > increasing to more than 2* the performance on larger memory blocks on > Graviton 2, betw

Re: RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory

2025-05-09 Thread Andrew Haley
On Fri, 9 May 2025 14:11:27 GMT, Andrew Haley wrote: > This intrinsic is generally faster than the current implementation for Panama > segment operations for all writes larger than about 8 bytes in size, > increasing to more than 2* the performance on larger memory blocks on > Graviton 2, betw

RFR: 8354674: AArch64: Intrinsify Unsafe::setMemory

2025-05-09 Thread Andrew Haley
This intrinsic is generally faster than the current implementation for Panama segment operations for all writes larger than about 8 bytes in size, increasing to more than 2* the performance on larger memory blocks on Graviton 2, between "panama" (C2 generated, what we use now) and "unsafe" (this