[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