Integrated: 7903878: jextract should derive struct field offsets from struct layouts

Nizar Benalla nbenalla at openjdk.org
Wed Dec 11 16:51:27 UTC 2024


On Fri, 6 Dec 2024 23:03:47 GMT, Nizar Benalla <nbenalla at openjdk.org> wrote:

> Please read the JBS issue for more details.
> 
> Currently jextract emits hardwired offsets, it would be better if the offsets could be computed dynamically from the struct layouts.
> 
> Passes all tests, unsure if this patch requires new tests.
> Note: I used `fieldElementPaths(field.name())))` instead of the `javaName` to deal with nested structs.
> 
> 
> It seems like GitHub actions are broken for Ubuntu? will look into this next week if necessary.

This pull request has now been integrated.

Changeset: 0614097d
Author:    Nizar Benalla <nbenalla at openjdk.org>
Committer: Maurizio Cimadamore <mcimadamore at openjdk.org>
URL:       https://git.openjdk.org/jextract/commit/0614097d45f01e43f6fe879f9dcc47bcac554788
Stats:     2 lines in 1 file changed: 0 ins; 0 del; 2 mod

7903878: jextract should derive struct field offsets from struct layouts

Reviewed-by: mcimadamore

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

PR: https://git.openjdk.org/jextract/pull/262


More information about the jextract-dev mailing list