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

2020-01-09 Thread Simon Tatham via Phabricator via cfe-commits
simon_tatham closed this revision. simon_tatham marked an inline comment as done. simon_tatham added a comment. Closed by rGd857e114b5e04f5143485a5aea7ad9b283768692 . (I managed to leave off the commit message footer, sorry.)

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

2020-01-09 Thread Dave Green via Phabricator via cfe-commits
dmgreen accepted this revision. dmgreen added a comment. This revision is now accepted and ready to land. Nice test. LGTM Comment at: clang/test/Sema/arm-mve-immediates.c:182 + + vshrq(vb, 0); // expected-error {{argument value 0 is outside the valid range [1, 8]}} + vshrq(v

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

2020-01-09 Thread 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,