[foreign-jextract] Integrated: 8253798: Simplify ClassConstantHelper

[Jorn Vernee]

On Tue, 29 Sep 2020 18:45:17 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

> Hi,
> 
> This patch removes a whole bunch of workarounds in jextract code, which are made possible by upstream changes to
> foreign-abi.
> ClassConstantHelper no longer has to manually canonicalize MemoryLayout constant that represent layout constants in
> CLinker. This is now handled by the CValueLayout.describeConstable() override.
> Making CValueLayout public also means we can remove the CanonicalField enum from jextract, and rely on the
> CValueLayout.Kind instead. (note that since the enum was internal, complex recursive filtering would be needed if we
> had kept it).  Thanks,
> Jorn

This pull request has now been integrated.

Changeset: 7c5851f7
Author:    Jorn Vernee <jvernee at openjdk.org>
URL:       https://git.openjdk.java.net/panama-foreign/commit/7c5851f7
Stats:     202 lines in 4 files changed: 8 ins; 178 del; 16 mod

8253798: Simplify ClassConstantHelper

Reviewed-by: mcimadamore

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

PR: https://git.openjdk.java.net/panama-foreign/pull/360