On Tue, 4 Feb 2025 18:55:25 GMT, Emanuel Peter <epe...@openjdk.org> wrote:
>> Suite MaskedLogicOpts.maskedLogicOperationsLong512() failed on both x86 and >> AArch64 with the following error: >> >> >> java.lang.IndexOutOfBoundsException: Index 252 out of bounds for length 249 >> >> >> The variable `long256_arr_idx` is misused when indexing 'LongVector l2, l3, >> l4, l5' in function `maskedLogicOperationsLongKernel()`. 'long256_arr_idx' >> increases by 4 every time the benchmark runs and ensures the incremented >> value remains within the bounds of the array. However, for >> `LongVector.SPECIES_512`, it loads 8 numbers from the array each time the >> benchmark runs, resulting in an out-of-range indexing issue. >> >> Hence, we revised the index variables from `long256_arr_idx` to >> `long512_arr_idx`, which has a stride of 8, to ensure that the loaded vector >> is inside of the array boundary for all vector species. This is also >> consistent with other kernel functions. >> >> Additionally, some defined but unused variables have been removed. > > Oh, the OCA-verify is still stuck. I'm sorry about that 🙈 > I pinged my manager @TobiHartmann , he will reach out to see what's the issue. Thanks @eme64. The OCA finally passed! 😄 ------------- PR Comment: https://git.openjdk.org/jdk/pull/22963#issuecomment-2646741800
