RFR: 8257600: [type-restrictions] Implement RestrictedField in C1

Frederic Parain fparain at openjdk.java.net
Wed Dec 2 14:19:17 UTC 2020

Please review theses changes adding RestrictedField support to C1.
Tests have been added using the annotation added by JDK-8255856.
Tests revealed some issues with the support of RestrictedField in the interpreter, fixes are included in this patch.
The changes also includes some renaming after discussions with Dan.

Thank you,



Commit messages:
 - Re-enable RestrictedType for static fields, minor fixes
 - Reject RestrictedField attribute on static fields
 - Fix tabs
 - Fix interpreter withfield, fix test 5, automated tests
 - More tests
 - Fix InstanceKlass size computation and add test
 - Fix code after RestrictedType model update
 - Merge remote-tracking branch 'upstream/type-restrictions' into restricted_fields_c1
 - Merge remote-tracking branch 'upstream/type-restrictions' into restricted_fields_c1
 - Add checkcasts to write access to restricted fields
 - ... and 1 more: https://git.openjdk.java.net/valhalla/compare/4902cf9c...21ac7562

Changes: https://git.openjdk.java.net/valhalla/pull/288/files
 Webrev: https://webrevs.openjdk.java.net/?repo=valhalla&pr=288&range=00
  Issue: https://bugs.openjdk.java.net/browse/JDK-8257600
  Stats: 489 lines in 17 files changed: 455 ins; 5 del; 29 mod
  Patch: https://git.openjdk.java.net/valhalla/pull/288.diff
  Fetch: git fetch https://git.openjdk.java.net/valhalla pull/288/head:pull/288

PR: https://git.openjdk.java.net/valhalla/pull/288

More information about the valhalla-dev mailing list