[foreign] RFR 8217414: Remove bounds check in BoundedPointer constructor
Jorn Vernee
jbvernee at xs4all.nl
Mon Jan 21 12:00:32 UTC 2019
Hi,
I have a patch addressing 2 vacuously passing tests (api/PointerTest and
api/ArrayTest). The patch fixes the tests, and to make that happen
removes the bounds check in BoundedPointer's constructor. As discussed
offline; this check is not needed since we already de a check when
dereferencing a pointer in BoundedMemoryRegion, and the check in the
constructor is overly restrictive since it should be fine to create
out-of-bounds pointers as long as it is not dereferenced. This does mean
the tests needed to be tweaked since now an access exception is throw at
a later point.
Please review the following.
Bug: https://bugs.openjdk.java.net/browse/JDK-8217414
Webrev:
http://cr.openjdk.java.net/~jvernee/panama/webrevs/8217414/webrev.00
Thanks,
Jorn
More information about the panama-dev
mailing list