[PATCH] D30329: [AArch64InstPrinter] rewrite of printSysAlias

Sjoerd Meijer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 24 03:06:24 PST 2017


SjoerdMeijer created this revision.
Herald added a subscriber: aemerson.

This is a cleanup/rewrite of the printSysAlias function. This was not using the tablegen instruction descriptions, but was "manually" decoding the instructions. This has been replaced with calls to lookup_XYZ_ByEncoding tablegen calls.

      

This revealed several problems. First, instruction IVAU had the wrong encoding. This was cancelled out by the parser that incorrectly matched the wrong encoding. Second, instruction CVAP was missing  from the SystemOperands tablegen descriptions, so this has been added. And third, the required target features were not captured in the tablegen descriptions, so support for this has also been added.


https://reviews.llvm.org/D30329

Files:
  lib/Target/AArch64/AArch64SystemOperands.td
  lib/Target/AArch64/InstPrinter/AArch64InstPrinter.cpp
  lib/Target/AArch64/Utils/AArch64BaseInfo.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D30329.89635.patch
Type: text/x-patch
Size: 13026 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170224/705b63f5/attachment.bin>


More information about the llvm-commits mailing list