[PATCH] D62667: [ARM] Add the non-MVE instructions in Arm v8.1-M.

Sam Parker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 31 03:13:00 PDT 2019


samparker added inline comments.


================
Comment at: llvm/lib/Target/ARM/ARMInstrVFP.td:2687
+                !con(oops, (outs GPRnopc:$wb)),
+                !con(iops, (ins t2_addr_offset_none:$Rn,
+                                t2am_imm7s4_offset:$addr)),
----------------
samparker wrote:
> Why does the post form have one more operand than the pre? This doesn't really look like the existing pre/post load/store instructions.
Scrap that, it does look like the existing ones but it still doesn't make sense to me. I'll have a closer look at the original descriptions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D62667





More information about the llvm-commits mailing list