[jdk20] RFR: 8299733: AArch64: "unexpected literal addressing mode" assertion failure with -XX:+PrintC1Statistics [v2]

[Andrew Haley]

On Fri, 6 Jan 2023 21:12:56 GMT, Dean Long <dlong at openjdk.org> wrote:

> Isn't the real problem that _base and _index are not always initialized? If modes that don't use those registers always initialized them to noreg, at least in debug builds, then wouldn't that allow the existing uses() to work without extra overhead in product mode?

I don't think it's necessarily a problem if base and _index are not always initialized, but if they aren't then they definitely shouldn't ever be read. I don't think we can rely on _base and _index being initialized, because Address contains a union, so may not even have those fields. The most robust fix is probably to make base() and index() return noreg if the mode of the Address instance doesn't have them set.

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

PR: https://git.openjdk.org/jdk20/pull/87