RFR: 8356993: ArrayDeque should use Arrays.fill() instead of for() loops

Roger Riggs rriggs at openjdk.org
Thu May 15 14:48:54 UTC 2025


On Wed, 14 May 2025 19:37:37 GMT, Archie Cobbs <acobbs at openjdk.org> wrote:

> Please review this small performance tweak `ArrayDeque`.
> 
> `ArrayDeque` has an invariant in which any unused elements in the array must be null. In a couple of places, the code is setting contiguous ranges of elements to null using `for()` loops. This can be both simplified and sped up by using `Arrays.fill()` instead.

I'm curious to know whether C2 turns the loop into a vectorized operation.  The Arrays.fill might be more expressive, but not necessarily faster.

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

PR Comment: https://git.openjdk.org/jdk/pull/25237#issuecomment-2884095477


More information about the core-libs-dev mailing list