Aarch64: emit prefix/postfix-indexed memory accesses

[igogo-x86]

• teach Vgen to generate prefix and postfix indexed forms for scalar/pair memory ops
• validate update immediates before encoding the indexed memory instructions
• let psimplify fold plain memory accesses into the new prefix/postfix variants

[meta-codesync]

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this in D86516435. (Because this pull request was imported automatically, there will not be any future comments.)

[meteorfox]

Thanks @igogo-x86 !

This one is failing with

hphp/runtime/vm/jit/vasm-simplify-arm.cpp:396:15: error: unused variable 'inst' [-Werror,-Wunused-variable]
  396 |   auto const& inst = env.unit.blocks[b].code[i];
      |               ^~~~
hphp/runtime/vm/jit/vasm-simplify-arm.cpp:415:15: error: unused variable 'inst' [-Werror,-Wunused-variable]
  415 |   auto const& inst = env.unit.blocks[b].code[i];
      |               ^~~~
2 errors generated.

[facebook-github-bot]

@igogo-x86 has updated the pull request. You must reimport the pull request before landing.

[igogo-x86]

Fixed!

[meteorfox]

New error

Stderr:
fbcode/hphp/runtime/vm/jit/vasm-info.cpp:159:11: error: 44 enumeration values not handled in switch: 'loadpri', 'loadpi', 'loadbpri'... [-Werror,-Wswitch]
  159 |   switch (inst.op) {
      |           ^~~~~~~
1 error generated.

[facebook-github-bot]

@igogo-x86 has updated the pull request. You must reimport the pull request before landing.

[meteorfox]

Hey @igogo-x86 , now fails to compile with this error

Stderr:
fbcode/hphp/runtime/vm/jit/vasm-instr.cpp:94:11: error: 44 enumeration values not handled in switch: 'loadpri', 'loadpi', 'loadbpri'... [-Werror,-Wswitch]
   94 |   switch (op) {
      |           ^~
1 error generated.

[facebook-github-bot]

@igogo-x86 has updated the pull request. You must reimport the pull request before landing.