[PATCH] D69716: NeonEmitter: remove special 'a' modifier.

Tim Northover via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 1 07:58:32 PDT 2019


t.p.northover created this revision.
t.p.northover added a reviewer: efriedma.
Herald added a subscriber: mcrosier.
Herald added a project: clang.

This has been separated off from D69618 <https://reviews.llvm.org/D69618> to reduce diff clutter, as it really stands on its own. The 'a' modifier is rarely used, but emits splatting code from NeonEmitter.cpp when it is. This functionality can be implemented directly in .td files though, at a net reduction in lines of code.

There are a few test changes, but they're just moving bitcasts around due to the slightly different input Clang sees. Nothing really changes.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D69716

Files:
  clang/include/clang/Basic/arm_neon.td
  clang/include/clang/Basic/arm_neon_incl.td
  clang/test/CodeGen/aarch64-neon-2velem.c
  clang/test/CodeGen/arm_neon_intrinsics.c
  clang/utils/TableGen/NeonEmitter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69716.227449.patch
Type: text/x-patch
Size: 46681 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20191101/cfb10a45/attachment-0001.bin>


More information about the cfe-commits mailing list