[PATCH] D72444: [ARM,MVE] Fix valid immediate range for vsliq_n.

[Simon Tatham via Phabricator via cfe-commits]

simon_tatham created this revision.
simon_tatham added reviewers: dmgreen, MarkMurrayARM, miyuki, ostannard.
Herald added subscribers: cfe-commits, kristof.beyls.
Herald added a project: clang.

In common with most MVE immediate shift instructions, the left shift
takes an immediate in the range [0,n-1], while the right shift takes
one in the range [1,n]. I had absent-mindedly made them both the
latter.

While I'm here, I've added a set of regression tests checking both
ends of the immediate range for a representative sample of the
immediate shifts.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D72444

Files:
  clang/include/clang/Basic/arm_mve.td
  clang/test/Sema/arm-mve-immediates.c

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D72444.237025.patch
Type: text/x-patch
Size: 5852 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200109/69613248/attachment-0001.bin>