On Thu, 19 Mar 2026 16:57:22 GMT, Roland Westrelin <[email protected]> wrote:

>> I think if the difference of the heap size with and without your fix is 
>> large, we can just pick a large enough value for the assert that would then 
>> trigger without your fix and still be large enough to account for 
>> fluctuations.
>
> To avoid timeouts, I decreased the number of iterations and the test runs 
> with `-Xmx20M`. I think it reports `2MB` being used with my patch. So if 
> memory usage increases for some reason, it's very likely to cause an OOME. Do 
> you still think an assert is needed?

That sounds good.

-------------

PR Review Comment: 
https://git.openjdk.org/valhalla/pull/2062#discussion_r3014429833

Reply via email to