RFR: 8259851: Using boolean type for tasks in SubTasksDone
Kim Barrett
kbarrett at openjdk.java.net
Thu Jan 21 10:15:53 UTC 2021
On Mon, 18 Jan 2021 20:36:43 GMT, Albert Mingkun Yang <ayang at openjdk.org> wrote:
>> Should this change be made? I understand the intent is to use the
>> semantically intended type, and agree with that intent. But there is a
>> hidden cost; some platforms don't directly support cmpxchg on byte sized
>> values, and use CmpxchgByteUsingInt. Maybe that cost is in the noise, but
>> the question should be considered. For Zero I don't care. But there are
>> affected platforms.
>
>> For Zero I don't care. But there are affected platforms.
>
> I see arm and s390 with grepping `CmpxchgByteUsingInt`; I will test specjbb2015 and dacapo on arm to see if there is any perf diff.
Discussed with @albertnetymk and agreed the impact of this is not going to be measurable, so withdrawing the question.
-------------
PR: https://git.openjdk.java.net/jdk/pull/2131
More information about the hotspot-gc-dev
mailing list