RFR: JDK-8254189: Improve comments for StackOverFlow and fix in_xxx() functions [v2]
Thomas Stuefe
stuefe at openjdk.java.net
Wed Oct 28 16:30:49 UTC 2020
On Wed, 28 Oct 2020 15:27:43 GMT, Gerard Ziemski <gziemski at openjdk.org> wrote:
>> Thomas Stuefe has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Feedback David
>
> Marked as reviewed by gziemski (Committer).
> Very nice.
Thank you, Gerard.
>
> Personally I'd like to see additional:
>
> ASSERT_TRUE(so.in_stack_red_zone(so.stack_end()));
> ASSERT_TRUE(so.in_stack_yellow_zone(so.stack_red_zone_base()));
> ASSERT_TRUE(so.in_stack_reserved_zone(so.stack_yellow_zone_base()));
>
> in the test, to drive home the point of the inclusion of the ranges, but that's a tiny nick pick.
I can do this. Will have to reshape those a bit since there is no "in_yellow_zone" (I considered adding one for tests sake but then did not).
...Thomas
-------------
PR: https://git.openjdk.java.net/jdk/pull/795
More information about the hotspot-runtime-dev
mailing list