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

[Fei Gao]

On Fri, 31 May 2024 15:13:44 GMT, Fei Gao <fgao at openjdk.org> wrote:

> However, I don't think that all the IndOffXX types do us any good. It would be simpler and faster to match a general-purpose IndOff type then let `legitimize_address()` fix any out-of-range operands. That'd reduce the size of the match rules and the time it takes to run them.

Hi @theRealAph @dean-long , the new commit implemented the scheme discussed before. It keeps two general operands `immIOffset` and `immLOffset` and lets `legitimize_address()` fix any out-of-range operands for different memory operations. To avoid confusion, it also keeps `indOffI` and `indOffIN` as separate operands.

Could you help review it please? Thanks!

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

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