On Wed, 11 Nov 2020 16:16:19 GMT, Erik Österlund <eosterl...@openjdk.org> wrote:
>> @coleenp - Nice catch on the missing 'static'. > > I typically use size_t for entities that can scale with the size of the > machine's memory, so I don't have to think about whether there are enough > bits. Could AvgMonitorsPerThreadEstimate be uintx instead of intx? And then > maybe we don't need to declare a range, as the natural range of the uintx > seems perfectly valid. I'm pretty sure I copied the decl for AvgMonitorsPerThreadEstimate from some other already existing option. That's SOP for me anyway... If we make any more changes here it will have to be in a follow up. ------------- PR: https://git.openjdk.java.net/jdk/pull/642
