Integrated: 8272315: Improve assert_different_registers

[Kim Barrett]

On Wed, 11 Aug 2021 11:34:26 GMT, Kim Barrett <kbarrett at openjdk.org> wrote:

> Please review this improvement to assert_different_register.
> 
> It reduces ~250 lines of repetitive code to ~10 lines, and improves the
> error message on failure by printing a duplicated registers name.
> 
> Unfortunately, this isn't enough to eliminate AbstractRegister[Impl].
> Although it seems to no longer be needed for the Oracle-supported platforms,
> the linux-arm/ppc64le/s390 platforms all use the value() function provided
> by AbstractRegisterImpl.
> 
> Testing:
> tier1 on Oracle supported platforms.
> built linux-arm/ppc64le/s390.
> 
> Locally changed a call to have a duplicate register and verified the error
> message had a useful register name.

This pull request has now been integrated.

Changeset: 93cab7d0
Author:    Kim Barrett <kbarrett at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/93cab7d07d56fef6973ce8ac2e8232e8f3b185dd
Stats:     257 lines in 1 file changed: 0 ins; 246 del; 11 mod

8272315: Improve assert_different_registers

Reviewed-by: adinn, vlivanov, thartmann

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

PR: https://git.openjdk.java.net/jdk/pull/5083