[lworld] [Rev 01] RFR: 8242612: [lworld] Javac should not expressly encode the new super interface types in class files

Srikanth Adayapalam sadayapalam at openjdk.java.net
Mon May 18 13:48:51 UTC 2020


> Jim, Could you please review this change ? TIA.
> 
> Changes:
> 
>     - Withdraw the new top type InlineObject (per design)
>     - Do not encode the new top types in class files as this is noisy and
>       very disruptive and let the static compiler and the dynamic environments
>       inject these types as needed. (since classes compiled with legacy compilers
>       will not encode these top types anyway the VM has to resort to injection,
>       why not resort to injection for everything is the motivation)
>     - Adjust some code that is surprised by super interfaces being empty.
> 
> More background is available in https://bugs.openjdk.java.net/browse/JDK-8242612

Srikanth Adayapalam has updated the pull request incrementally with one additional commit since the last revision:

  Simplify code per review comments.

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

Changes:
  - all: https://git.openjdk.java.net/valhalla/pull/50/files
  - new: https://git.openjdk.java.net/valhalla/pull/50/files/94e232ee..ecb41ea4

Webrevs:
 - full: https://webrevs.openjdk.java.net/valhalla/50/webrev.01
 - incr: https://webrevs.openjdk.java.net/valhalla/50/webrev.00-01

  Stats: 14 lines in 2 files changed: 0 ins; 10 del; 4 mod
  Patch: https://git.openjdk.java.net/valhalla/pull/50.diff
  Fetch: git fetch https://git.openjdk.java.net/valhalla pull/50/head:pull/50

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


More information about the valhalla-dev mailing list