RFR: 7903146: TypeTranslator is no longer needed [v3]

Maurizio Cimadamore mcimadamore at openjdk.java.net
Wed Apr 6 16:08:25 UTC 2022


> Now that ValueLayout has carrier information attached, we no longer need to perform two separate visit on a function type, one to get a FunctionDescriptor and one to get a MethodType. We can just get a FunctionDescriptor and derive everything from there.
> 
> This patch applies several cleanups on OutputFactory, to flatten the code, and remove some spaghetti lambdas. It also drops the FunctionInfo/VarInfo abstraction, since now that we can derive most of the info from layouts, these do not seem to add much value.

Maurizio Cimadamore has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 10 commits:

 - Merge branch 'master' into drop_type_translator
 - Update copyright
 - Drop VarInfo
 - Drop FunctionINfo
 - Initial push
 - Merge branch 'cleanup_treemaker' into drop_type_translator
 - Address review comments
 - Drop TypeTranslator
 - Merge branch 'master' into cleanup_treemaker
 - Initial push

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

Changes: https://git.openjdk.java.net/jextract/pull/14/files
 Webrev: https://webrevs.openjdk.java.net/?repo=jextract&pr=14&range=02
  Stats: 360 lines in 10 files changed: 38 ins; 214 del; 108 mod
  Patch: https://git.openjdk.java.net/jextract/pull/14.diff
  Fetch: git fetch https://git.openjdk.java.net/jextract pull/14/head:pull/14

PR: https://git.openjdk.java.net/jextract/pull/14


More information about the jextract-dev mailing list