RFR: Add aarch64 specific lock/unlock tsan instruments
Arthur Eubanks
aeubanks at openjdk.java.net
Fri May 8 16:21:17 UTC 2020
On Fri, 8 May 2020 03:44:13 GMT, Jie He <github.com+10233373+jhe33 at openjdk.org> wrote:
>>> What do you think about making the `16` based on the arch?
>>>
>>> `assert((id_u8 & left_n_bits(16)) == 0, "jmethodID is not aligned");`
>>
>> the assertion guarantees it's a 47-bit address, I think it's not neccessary for aarch64, I can add a AMD64_ONLY for it.
>
>> > What do you think about making the `16` based on the arch?
>> > `assert((id_u8 & left_n_bits(16)) == 0, "jmethodID is not aligned");`
>>
>> the assertion guarantees it's a 47-bit address, I think it's not neccessary for aarch64, I can add a AMD64_ONLY for it.
>
> done
LGTM
-------------
PR: https://git.openjdk.java.net/tsan/pull/9
More information about the tsan-dev
mailing list