[lworld] Integrated: 8281283: Remove usages of __WithField in runtime tests

Dan Smith dlsmith at openjdk.java.net
Wed Feb 23 23:36:29 UTC 2022


On Thu, 17 Feb 2022 00:56:50 GMT, Dan Smith <dlsmith at openjdk.org> wrote:

> Modifying Valhalla runtime tests to no longer use `__WithField` as a macro for the `withfield` instruction. Most uses can be replaced with standard constructor compilation via javac. The remaining uses are rewritten in jasm.
> 
> Summary of changes:
> 
> - Remove all uses of `-XDallowWithFieldOperator`
> - Replace various `make` and `create` factory methods for primitive classes with constructors
> - Bug fixes for `JumboInline` that were exposed by this change
> - Rewrote `TestFieldNullability` to use jasm
> - Rewrote `VWithFieldTest` to use jasm, renamed `WithFieldTest`
> - Rewrote `WithFieldAccessorTest` to use jasm, made more comprehensive
> - Deleted `WithFieldNoAccessTest`, made redundant by `WithFieldAccessorTest` changes
> - Minor tweaks to `TestFieldTypeMismatch` for consistent jasm usage
> - Cleanup of some debugging output in `asmtools.JtregDriver`

This pull request has now been integrated.

Changeset: 71cf8628
Author:    Dan Smith <dlsmith at openjdk.org>
URL:       https://git.openjdk.java.net/valhalla/commit/71cf8628fa3133ff43bff0238f7a4eaa921cb88d
Stats:     2016 lines in 38 files changed: 703 ins; 1105 del; 208 mod

8281283: Remove usages of __WithField in runtime tests

Reviewed-by: rriggs, dsimms

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

PR: https://git.openjdk.java.net/valhalla/pull/653



More information about the valhalla-dev mailing list