[PATCH] D69790: [ARM,MVE] Integer-type nitpicks in MVE intrinsics.

Simon Tatham via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 4 01:54:06 PST 2019


simon_tatham created this revision.
simon_tatham added a reviewer: dmgreen.
Herald added subscribers: cfe-commits, kristof.beyls.
Herald added a project: clang.
simon_tatham added a child revision: D69791: [ARM,MVE] Add intrinsics for gather/scatter load/stores..

A few integer types in the ACLE definitions of MVE intrinsics are
given as 'int' or 'unsigned' instead of <stdint.h> fixed-size types
like uint32_t. Usually these are the ones where the size isn't that
important, such as immediate offsets in loads (which have a range
limited by the instruction encoding) or the carry flag in vadcq which
can only be 0 or 1 anyway.

With this change, <arm_mve.h> follows that exact type naming, so that
the function prototypes look identical to the ones in ACLE, instead of
replacing int and unsigned with int32_t and uint32_t.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D69790

Files:
  clang/include/clang/Basic/arm_mve.td
  clang/include/clang/Basic/arm_mve_defs.td
  clang/utils/TableGen/MveEmitter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69790.227659.patch
Type: text/x-patch
Size: 6440 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20191104/916788c3/attachment.bin>


More information about the cfe-commits mailing list