RFR (L): 8060025: Object copy time regressions after JDK-8031323 and JDK-8057536
kim.barrett at oracle.com
Thu Dec 18 23:46:18 UTC 2014
On Dec 15, 2014, at 7:03 AM, Thomas Schatzl <thomas.schatzl at oracle.com> wrote:
>> 88 assert(index < InCSetState::Num,
>> Instead of InCSetState::Num, consider
>> sizeof(_dest) / sizeof(_dest)
>> And maybe we should add the usual array_size() utility to
>> globalDefinitions.hpp. A quick search of hotspot sources shows many
>> places where it would be appropriate to use.
> This has been removed in the current change.
> I would prefer to create a separate CR for introducing an array_size()
FYI, it turns out we already have ARRAY_SIZE() in globalDefinitions.hpp,
although not with the best definition for C++. It was there at the dawn of
the mercurial age, though presently only used in about a dozen places
(out of *many* more where it would be appropriate).
More information about the hotspot-gc-dev