[PATCH] D60701: [ARM] add new LLVM addressing modes for v8.1-M and MVE.

[Oliver Stannard (Linaro) via Phabricator via llvm-commits]

ostannard added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMInstrFormats.td:112
 def AddrModeT2_ldrex : AddrMode<18>;
+def AddrMode_bf     : AddrMode<19>;
+def AddrModeT2_i7s4 : AddrMode<20>;
----------------
simon_tatham wrote:
> t.p.northover wrote:
> > Why no T2?
> I think just because there's no BF in any other instruction set to need to distinguish it from. I can add "T2" in the name anyway if you prefer.
Do we actually need an AddrMode for BF? All of the uses of them I've seen are for loads and stores, and I don't think we have any AddrModes for existing branches.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60701/new/

https://reviews.llvm.org/D60701