[foreign-abi] Integrated: Copy stubs from Compile arena to nmethod stub array

Jorn Vernee jvernee at openjdk.java.net
Thu Jul 2 17:34:16 UTC 2020


On Thu, 2 Jul 2020 14:39:07 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

> Hi,
> 
> This is a small followup on the intrinsics patch (https://github.com/openjdk/panama-foreign/pull/219).
> 
> @slowhog was seeing a crash when native invoker stubs were being freed, and it turned out that this was due to not
> copying the stubs properly from the compile arena to the nmethod stub array when creating the nmethod. This patch fixes
> that.  I had wrongly assumed that this code path would be exercised by the current tests, but this is evidently not the
> case. I'll think of a way to add a test for this specific case, but in the mean time this is a stop-gap fix to resolve
> the crash.  Thanks,
> Jorn

This pull request has now been integrated.

Changeset: e84922ec
Author:    Jorn Vernee <jvernee at openjdk.org>
URL:       https://git.openjdk.java.net/panama-foreign/commit/e84922ec
Stats:     3 lines in 1 file changed: 0 ins; 3 del; 0 mod

Copy stubs from Compile arena to nmethod stub array

Reviewed-by: henryjen

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

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


More information about the panama-dev mailing list