[PATCH] D63492: [ARM] Rename MVE instructions in Tablegen for consistency.

Simon Tatham via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 18 06:36:27 PDT 2019


simon_tatham created this revision.
simon_tatham added reviewers: ostannard, SjoerdMeijer, dmgreen.
Herald added subscribers: llvm-commits, hiraditya, kristof.beyls, javed.absar.
Herald added a project: LLVM.

Their names began with a mishmash of `MVE_`, `t2` and no prefix at
all. Now they all start with `MVE_`, which seems like a reasonable
choice on the grounds that (a) NEON is the thing they're most at risk
of being confused with, and (b) MVE implies Thumb-2, so a prefix
indicating MVE is strictly more specific than one indicating Thumb-2.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D63492

Files:
  llvm/lib/Target/ARM/ARMBaseInstrInfo.h
  llvm/lib/Target/ARM/ARMInstrMVE.td
  llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp
  llvm/lib/Target/ARM/Disassembler/ARMDisassembler.cpp
  llvm/lib/Target/ARM/Thumb2ITBlockPass.cpp
  llvm/test/CodeGen/ARM/mve-vpt-block.mir

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D63492.205323.patch
Type: text/x-patch
Size: 37348 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190618/c82487f6/attachment.bin>


More information about the llvm-commits mailing list