Integrated: 7903146: TypeTranslator is no longer needed

Maurizio Cimadamore mcimadamore at openjdk.java.net
Wed Apr 6 16:51:02 UTC 2022


On Wed, 6 Apr 2022 15:18:05 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:

> 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.

This pull request has now been integrated.

Changeset: b3028ceb
Author:    Maurizio Cimadamore <mcimadamore at openjdk.org>
URL:       https://git.openjdk.java.net/jextract/commit/b3028ceb6dd756de5130988a476566f46b3aa136
Stats:     360 lines in 10 files changed: 38 ins; 214 del; 108 mod

7903146: TypeTranslator is no longer needed

Reviewed-by: sundar

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

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


More information about the jextract-dev mailing list