RFR: 8346954: [JMH] jdk.incubator.vector.MaskedLogicOpts fails due to IndexOutOfBoundsException [v2]

[Nicole Xu]

On Tue, 25 Feb 2025 00:00:23 GMT, Paul Sandoz <psandoz at openjdk.org> wrote:

>> Thanks for pointing that out. Typically, ARRAYLEN is almost always a POT value, which is also assumed by many other benchmarks. Are we realistically going to test with an ARRAYLEN of 30? 
>> 
>> I think the POT assumption is reasonable for our purposes.
>
> It's a reasonable assumption. Since `ARRAYLEN` is a parameter of the benchmark we should enforce that constraint in benchmark initialization method, checking if the value is POT and failing otherwise.

Hi, @PaulSandoz, thanks for your suggestions.

In @jatin-bhateja's latest updates, non-POT `ARRAYLEN` values have been supported by processing only the elements up to the nearest multiple of the vector length. In that case, we can use a wider range of `ARRAYLEN` values while focusing on the main loop for vector functionality.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22963#discussion_r1971372152