On Thu, 11 Jun 2026 20:10:08 GMT, Kevin Walls <[email protected]> wrote:

>> Chris Plummer has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   get rid of unneeded module
>
> Looks good.
> We're checking "hr.top() == hrTop.top()" which compares top() of the HR we 
> got from the iterator, with hrTop.top() which is a HR we got by address, from 
> hr.top() or possibly from hr.top() - 1. 
> The regions should just be completely the same?  But we don't need to add a 
> full comparison for this if we don't have it already.

Thank you @kevinjwalls and @YaSuenag for the reviews!

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

PR Comment: https://git.openjdk.org/jdk/pull/31469#issuecomment-4710058767

Reply via email to