July 2023 - jextract-dev

RFR: 7903239: ofAddress factory of function pointer type is wrong for struct returns
by Marko 02 Jan '24

02 Jan '24

When generating the lambda inside the `ofAddress` factory, it is missing the `SegmentAllocator` needed when the function returns a struct. This is [implemented for regular functions](https://github.com/openjdk/jextract/blob/0582eaf1b4cdba95f0ee8c2…, but not for function pointers. ------------- Commit messages: - 7903239: ofAddress factory of function pointer type is wrong for struct returns Changes: https://git.openjdk.org/jextract/pull/58/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=58&range=00 Issue: https://bugs.openjdk.org/browse/CODETOOLS-7903239 Stats: 47 lines in 3 files changed: 44 ins; 2 del; 1 mod Patch: https://git.openjdk.org/jextract/pull/58.diff Fetch: git fetch https://git.openjdk.org/jextract pull/58/head:pull/58 PR: https://git.openjdk.org/jextract/pull/58

3 11

RFR: Add path independence
by Per Minborg 02 Jan '24

02 Jan '24

This small PR incorporated feedback from the original PR: https://github.com/openjdk/jextract/pull/66 made after it was integrated. ------------- Commit messages: - Add path independence Changes: https://git.openjdk.org/jextract/pull/68/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=68&range=00 Stats: 10 lines in 4 files changed: 2 ins; 0 del; 8 mod Patch: https://git.openjdk.org/jextract/pull/68.diff Fetch: git fetch https://git.openjdk.org/jextract pull/68/head:pull/68 PR: https://git.openjdk.org/jextract/pull/68

5 7

RFR: Add Dockerfile, zero-dep reproducible builds
by Gavin Ray 02 Jan '24

02 Jan '24

As title says, adds a Dockerfile that will fetch all necessary deps and build `jextract` for you on Ubuntu 22.10 It would be nice if some official JDK organization could put this on their Dockerhub, so that users could just do: $ docker run --rm -it openjdk/jextract:latest I've pushed it to my personal Dockerhub, so if anyone would like to test it out, you can use the README directions but prepend `gavinray/` before the image name: $ echo "struct Foo { int a; long b; }; int use_foo(struct Foo foo);" >> sample.h $ docker run --rm -it -v ${PWD}:/tmp/jextract \ gavinray/jextract:latest \ -I /tmp/jextract \ --source \ -l libfoo \ --output /tmp/jextract/generated \ --target-package com.example \ ------------- Commit messages: - Add Dockerfile, zero-dep reproducible builds Changes: https://git.openjdk.org/jextract/pull/74/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=74&range=00 Stats: 75 lines in 2 files changed: 75 ins; 0 del; 0 mod Patch: https://git.openjdk.org/jextract/pull/74.diff Fetch: git fetch https://git.openjdk.org/jextract pull/74/head:pull/74 PR: https://git.openjdk.org/jextract/pull/74

3 6

RFR: Add third-party sample list to README, populate with java-vulkan.
by Michael Ennen 02 Jan '24

02 Jan '24

I made a more complex example (1000 lines, mostly because setting up a boilerplate Vulkan program is super lengthy) of using jextract on Vulkan and Win32 APIs. I think it may be too lengthy and too much of a maintenance burden to commit to the `/samples` directory, so I thought maybe including a section on third-party samples would be better. If this is not something you want to entertain that's totally understandable, but thought I would put it out there a bigger sample that interfaces with a pretty complex API may be helpful to others. ------------- Commit messages: - Add third-party sample list to README, populate with java-vulkan. Changes: https://git.openjdk.org/jextract/pull/93/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=93&range=00 Stats: 3 lines in 1 file changed: 2 ins; 0 del; 1 mod Patch: https://git.openjdk.org/jextract/pull/93.diff Fetch: git fetch https://git.openjdk.org/jextract pull/93/head:pull/93 PR: https://git.openjdk.org/jextract/pull/93

3 3

jextract C++ support
by Rel 24 Sep '23

24 Sep '23

Hi, I would like to know how to participate in C++ support for jextract. Watching Project Panama video (https://inside.java/2023/04/18/levelup-panama/) Paul mentioned that C++ is in the plans. Do we have someone working on it already so I can syncup on what is the plan and where I can help? In particular: - will it be part of jextract or may be jextract++? - will it use clang or something else? if clang then which interface https://clang.llvm.org/docs/Tooling.html There are many things to be done for C++ support but if I pick the most basic like symbols, in C++ they are mangled so current jextract linking logic will need to be changed. Do you think modifying NameMangler to store those mangled C++ symbols will be the right approach? Regards,

4 15

jlink error when building jextract on macOS for JDK 21
by Manuel Bleichenbacher 25 Jul '23

25 Jul '23

As part of upgrading https://github.com/manuelbl/JavaDoesUSB to JDK 21, I'm building jextract for JDK 21. It works without issue on Linux and Windows. But on macOS it fails with this error message: *> Task :createJextractImage* FAILED Error: jdk.tools.jlink.plugin.PluginException: Duplicate resources: {include/darwin/jawt_md.h=[jdk.compiler, jdk.internal.opt], conf/jaxp.properties=[jdk.compiler, jdk.internal.opt], include/jawt.h=[jdk.compiler, jdk.internal.opt]} >From my limited understanding of jlink, I think one of the explicitly specified modules has a transitive dependency to the module "jdk.internal.opt", so it's included as well. But the modules "jdk.compiler" and "jdk.internal.opt" have three common files, which are now in conflict. Is this a known issue? Is there a workaround? My environment is: - JDK for gradle: openjdk version "17.0.1" 2021-10-19 LTS - JDK 21: OpenJDK 64-Bit Server VM Zulu21+69-CA (build 21-ea+28, mixed mode, sharing) - jextract: head of "jdk21" branch - Command line: sh ./gradlew -Pjdk21_home=/Users/me/Documents/zulu21.0.69-ea-jdk21.0.0-ea.28-macosx_aarch64 -Pllvm_home=/Library/Developer/CommandLineTools/usr/ clean verify

2 5

RFR: samples fixes for JDK 21 panama API changes and version change.
by Athijegannathan Sundararajan 25 Jul '23

25 Jul '23

All samples fine on MacOS. I changed run_windows scripts but I've not tested samples on Windows. ------------- Commit messages: - trailing whitespace - samples fixes for JDK 21 panama API changes and version change. Changes: https://git.openjdk.org/jextract/pull/126/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=126&range=00 Stats: 224 lines in 30 files changed: 50 ins; 77 del; 97 mod Patch: https://git.openjdk.org/jextract/pull/126.diff Fetch: git fetch https://git.openjdk.org/jextract.git pull/126/head:pull/126 PR: https://git.openjdk.org/jextract/pull/126

3 4

RFR: sample fixes for JDK 22 Panama API changes.
by Athijegannathan Sundararajan 24 Jul '23

24 Jul '23

sample fixes for JDK 22 Panama API changes. All samples fine on MacOS. Windows scripts have to be adjusted and samples have to be tested on Windows. ------------- Commit messages: - sample fixes for JDK 22 Panama API changes. Changes: https://git.openjdk.org/jextract/pull/125/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=125&range=00 Stats: 309 lines in 78 files changed: 51 ins; 77 del; 181 mod Patch: https://git.openjdk.org/jextract/pull/125.diff Fetch: git fetch https://git.openjdk.org/jextract.git pull/125/head:pull/125 PR: https://git.openjdk.org/jextract/pull/125

3 5

RFR: Tweak jextract to work with latest API changes
by Maurizio Cimadamore 20 Jul '23

20 Jul '23

This PR fixes a bunch of issues with jextract that were introduced by the latest changes in the Panama repo: * All var handles now take an extra offset parameter. This required changes to the libclang generated bindings, as well as changes in the code generated by jextract. * The name of some allocation factories in SegmentAllocator has changed. This PR fixes jextract to use the right names. * ValueLayout::arrayElementVarHandle is no longer present ------------- Commit messages: - Tweak jextract to work with latest API changes Changes: https://git.openjdk.org/jextract/pull/124/files Webrev: https://webrevs.openjdk.org/?repo=jextract&pr=124&range=00 Stats: 41 lines in 15 files changed: 0 ins; 4 del; 37 mod Patch: https://git.openjdk.org/jextract/pull/124.diff Fetch: git fetch https://git.openjdk.org/jextract.git pull/124/head:pull/124 PR: https://git.openjdk.org/jextract/pull/124

2 3