[foreign-abi] Integrated: 8254156: Simplify ABI classification logic
Maurizio Cimadamore
mcimadamore at openjdk.java.net
Wed Oct 7 13:36:17 UTC 2020
On Wed, 7 Oct 2020 11:39:50 GMT, Maurizio Cimadamore <mcimadamore at openjdk.org> wrote:
> This patch reverts recent changes to introduce a public CValueLayout class; the realization here is that, after all,
> CValueLayout exposes a public `Type` enum which is used by ABI classification. So, instead of using subclassing (which
> is messy and tedious with immutable data structures with covariant overrides), let's just double down on layout
> attributes, get rid of CValueLayout and simply expose the Type enum (now renamed to TypeKind).
This pull request has now been integrated.
Changeset: 6663aabe
Author: Maurizio Cimadamore <mcimadamore at openjdk.org>
URL: https://git.openjdk.java.net/panama-foreign/commit/6663aabe
Stats: 318 lines in 9 files changed: 43 ins; 153 del; 122 mod
8254156: Simplify ABI classification logic
Reviewed-by: jvernee
-------------
PR: https://git.openjdk.java.net/panama-foreign/pull/371
More information about the panama-dev
mailing list