[foreign-memaccess+abi] Integrated: 8313894: Rename isTrivial linker option to critical

[Jorn Vernee]

On Mon, 7 Aug 2023 17:21:45 GMT, Jorn Vernee <jvernee at openjdk.org> wrote:

> The name `isTrivial` isn't ideal.
> 
> The 'is' can be dropped, and we'd like to switch to the name 'critical' which leans on an established meaning (JNI GetPrimitiveArrayCritical, java criticals).
> 
> All in all, the goal of the name is to avoid making too many promises about what this option does (although that is covered by the javadoc). It is intended to act more as a hint to the runtime. For instance, on x64 it will elide the thread state transition done by a native call, but this is not the case on all implementations (e.g. the fallback linker).

This pull request has now been integrated.

Changeset: ea82d410
Author:    Jorn Vernee <jvernee at openjdk.org>
URL:       https://git.openjdk.org/panama-foreign/commit/ea82d4101e7ed33fe99d9faa0be59fd88a8acffd
Stats:     334 lines in 13 files changed: 143 ins; 143 del; 48 mod

8313894: Rename isTrivial linker option to critical

Reviewed-by: pminborg, mcimadamore

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

PR: https://git.openjdk.org/panama-foreign/pull/859