This Message Is From an External Sender
This message came from outside your organization.
On Wed, 6 Mar 2024 13:43:00 GMT, Magnus Ihse Bursie <[email protected]> wrote:
>> Currently, our symbol visibility handling for tests are sloppy; we only handle it properly on Windows. We need to bring it up to the same levels as product code. This is a prerequisite for [JDK-8327045](https://bugs.openjdk.org/browse/JDK-8327045), which in turn is a building block for Hermetic Java.
>
> Magnus Ihse Bursie has updated the pull request incrementally with one additional commit since the last revision:
>
> Update line number for dereference_null in TestDwarf
Thanks, this looks good to me.
test/jdk/java/foreign/CallGeneratorHelper.java line 216:
> 214: if (header) {
> 215: System.out.println(
> 216: "#include \"export.h\"\n"
We don't generate these header files any more, so the changes to this file are not really needed.
-------------
Marked as reviewed by jvernee (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/18135#pullrequestreview-1925503653
PR Review Comment: https://git.openjdk.org/jdk/pull/18135#discussion_r1518086954
