[PATCH] D54633: [NFC][AArch64] Split out backend features

Diogo N. Sampaio via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 26 09:03:02 PST 2018


dnsampaio updated this revision to Diff 175279.
dnsampaio marked 10 inline comments as done.
dnsampaio added a comment.

Fixed:
 the features implications/dependencies.
 the name of the JavaScript conversion feature.
 some test files.

@olista01: Most GCC devs I asked about internal feature names emphatically said that sync of internal names of features is not required. And most of the features are not even visible internally, as they are also associated with architecture reviews. So not much to do about that.

So I tried to make their description as close as possible to the reference manual, as to make it easy to grep for them. Further changes for defining arguments should be straightforward.


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

https://reviews.llvm.org/D54633

Files:
  lib/Target/AArch64/AArch64.td
  lib/Target/AArch64/AArch64InstrFormats.td
  lib/Target/AArch64/AArch64InstrInfo.td
  lib/Target/AArch64/AArch64Subtarget.h
  lib/Target/AArch64/AArch64SystemOperands.td
  lib/Target/AArch64/AsmParser/AArch64AsmParser.cpp
  test/MC/AArch64/armv8.2a-at.s
  test/MC/AArch64/armv8.2a-persistent-memory.s
  test/MC/AArch64/armv8.3a-complex.s
  test/MC/AArch64/armv8.3a-js.s
  test/MC/AArch64/armv8.3a-signed-pointer.s
  test/MC/AArch64/armv8.4a-flag.s
  test/MC/AArch64/armv8.4a-ldst.s
  test/MC/AArch64/armv8.4a-tlb.s
  test/MC/AArch64/armv8.4a-trace.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D54633.175279.patch
Type: text/x-patch
Size: 89719 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20181126/f37b169e/attachment.bin>


More information about the llvm-commits mailing list