Re: RFR: 8350075: Performance difference between SegmentAllocator methods [v2]

2025-04-14 Thread Per Minborg
On Thu, 10 Apr 2025 06:00:48 GMT, Per Minborg wrote: >> This PR proposes to add `@ForceInline` to the `default` methods in >> `SegmentAllocator` that do not already have it. > > Per Minborg has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev

Re: RFR: 8350075: Performance difference between SegmentAllocator methods

2025-04-10 Thread Per Minborg
On Fri, 14 Feb 2025 09:19:35 GMT, Per Minborg wrote: > This PR proposes to add `@ForceInline` to the `default` methods in > `SegmentAllocator` that do not already have it. Base: Benchmark Mode Cnt Score Error Units AllocFromInlineTest.alloc00 thrpt

Re: RFR: 8350075: Performance difference between SegmentAllocator methods [v2]

2025-04-10 Thread Maurizio Cimadamore
On Thu, 10 Apr 2025 06:00:48 GMT, Per Minborg wrote: >> This PR proposes to add `@ForceInline` to the `default` methods in >> `SegmentAllocator` that do not already have it. > > Per Minborg has updated the pull request with a new target base due to a > merge or a rebase. The incremental webrev

Re: RFR: 8350075: Performance difference between SegmentAllocator methods [v2]

2025-04-09 Thread Per Minborg
> This PR proposes to add `@ForceInline` to the `default` methods in > `SegmentAllocator` that do not already have it. Per Minborg has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase

Re: RFR: 8350075: Performance difference between SegmentAllocator methods

2025-04-09 Thread Per Minborg
On Fri, 14 Feb 2025 09:19:35 GMT, Per Minborg wrote: > This PR proposes to add `@ForceInline` to the `default` methods in > `SegmentAllocator` that do not already have it. Keep it open. - PR Comment: https://git.openjdk.org/jdk/pull/23628#issuecomment-2790078845

Re: RFR: 8350075: Performance difference between SegmentAllocator methods

2025-02-18 Thread Per Minborg
On Fri, 14 Feb 2025 09:19:35 GMT, Per Minborg wrote: > This PR proposes to add `@ForceInline` to the `default` methods in > `SegmentAllocator` that do not already have it. There is [a microbenchmark from another PR](test/micro/org/openjdk/bench/java/lang/foreign/ArenaPoolFromBench.java) which

Re: RFR: 8350075: Performance difference between SegmentAllocator methods

2025-02-14 Thread Maurizio Cimadamore
On Fri, 14 Feb 2025 09:19:35 GMT, Per Minborg wrote: > This PR proposes to add `@ForceInline` to the `default` methods in > `SegmentAllocator` that do not already have it. Change is good. There should be a microbenchmark for this. - PR Review: https://git.openjdk.org/jdk/pull/2362