RFR: 8319690: [AArch64] C2 compilation hits offset_ok_for_immed: assert "c2 compiler bug" [v4]

[Fei Gao]

On Fri, 5 Jul 2024 09:19:47 GMT, Andrew Haley <aph at openjdk.org> wrote:

> > > We can still see the extra register copy. So, I guess it is not caused by this PR and it does exist before?
> > 
> > 
> > If so then might it be better to relocate Andrew's patch to a separate issue? We may want to backport it independent from this change.
> 
> And there may be a more powerful way to fix it than my suggestion. I'm thinking of something analgous to `iRegIorL2I` which may be used instead of `regP` as an input to all address-producing patterns.

Hi @theRealAph @adinn, I proposed two pull requests, https://github.com/openjdk/jdk/pull/20157 and https://github.com/openjdk/jdk/pull/20159, for the issue. https://github.com/openjdk/jdk/pull/20159 is a draft because I'm not sure if it's worthwhile. I'd appreciate it if you could help review them. Thanks!

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

PR Comment: https://git.openjdk.org/jdk/pull/16991#issuecomment-2225665246