[GR-50774] ByteArrayAccessCloneBenchmark#unsafeCopyMemory does not use ARRAY_BYTE_BASE_OFFSET

[Marcono1234]

Describe the issue
The class ByteArrayAccessCloneBenchmark#unsafeCopyMemory uses 0 as offset values for the Unsafe#copyMemory call:

graal/truffle/src/com.oracle.truffle.api.benchmark/src/com/oracle/truffle/api/benchmark/ByteArrayAccessCloneBenchmark.java

Line 90 in 0808c62

UNSAFE.copyMemory(array, 0, copy, 0, array.length);

I think this is incorrect; it should instead use Unsafe#ARRAY_BYTE_BASE_OFFSET¹. Otherwise the current code actually also copies the array header and misses the last elements. This is not only incorrect behavior, but it might also distort the benchmark since it is accessing a slightly different memory section than all other methods (?).

Footnotes

1. And technically it should maybe also multiply array.length by ARRAY_BYTE_INDEX_SCALE? But since that value is 1 it might not matter much. ↩

[selhagani]

Hi @Marcono1234, thank you for sharing this with us!
I will share this with our dev team and I will keep you updated.