RFR: JDK-8257588: Make os::_page_sizes a bitmask [v2]
Marcus G K Williams
github.com+168222+mgkwill at openjdk.java.net
Thu Dec 3 17:57:56 UTC 2020
On Thu, 3 Dec 2020 13:56:03 GMT, Thomas Stuefe <stuefe at openjdk.org> wrote:
>> To be really nitpicky: the subject of the PR should be made more specific to mention that the new set is a bitset not just a set. I.e. os::page_sizes has already previously been a set (of integers), so where's the change? :P
>
> Hi Thomas,
>
> thanks again for the review. I hope I addressed all points. I completely removed the comments from the cpp file and corrected them in the hpp file, which removes the need to keep them in sync. And I rather have them in the hpp file, that makes my IDE happy.
>
> Cheers, Thomas
+1 on the general thrust of this change.
I'd be happy to incorporate into https://github.com/openjdk/jdk/pull/1153 & JDK-8256155 [1] if they are approved.
[1] https://bugs.openjdk.java.net/browse/JDK-8256155
-------------
PR: https://git.openjdk.java.net/jdk/pull/1522
More information about the hotspot-dev
mailing list