On 25/04/2019 17:34, Andrew Dinn wrote:
Also, here is a new webrev including the updated implementations for
mappingAddress/Offset/Length as described below
JIRA: https://bugs.openjdk.java.net/browse/JDK-8221696
webrev: http://cr.openjdk.java.net/~adinn/8221696/webrev.02
This looks right and I agree with your notes on negative addresses. As
its tricky to get right then it would be good to get another set of eyes
on this.
A minor nit is that we don't usually use underscores in variable names
here, it would be baseAddress rather than base_address, etc.
-Alan