[PATCH] D62383: [AArch64][SVE2] Asm: support SVE2 Floating Point Pairwise Group

Cullen Rhodes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 24 07:52:15 PDT 2019


c-rhodes added inline comments.


================
Comment at: test/MC/AArch64/SVE2/faddp-diagnostics.s:36
+// --------------------------------------------------------------------------//
+// Predicate not in restricted predicate range
+
----------------
chill wrote:
> Also `p0/z` is invalid predicate, e.g.:
> 
> ```
> faddp z0.h, p0/z, z0.h, z1.h
> // CHECK: [[@LINE-1]]:{{[0-9]+}}: error: invalid operand
> ```
> etc.
> 
I can add a test for that before merging. Managed to miss this for quite a few of the predicate instructions it seems. I have a patch downstream that improves the SVE predicate tests also as similar cases are missing there, I'll get that up after the SVE2 stuff is in.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62383/new/

https://reviews.llvm.org/D62383





More information about the llvm-commits mailing list