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