[PATCH] D41819: [AArch64][SVE] Asm: PTRUE and PTRUES instructions

[Sam Parker via Phabricator via llvm-commits]

samparker added inline comments.


================
Comment at: test/MC/AArch64/SVE/ptrue-diagnostics.s:8
+ptrue p0.s, vl512
+// CHECK: [[@LINE-1]]:{{[0-9]+}}: error: invalid operand
+// CHECK-NEXT: ptrue p0.s, vl512
----------------
Can we not give the user a more specific error message?


================
Comment at: test/MC/AArch64/SVE/ptrue.s:175
+// ---------------------------------------------------------------------------//
+
+ptrue   p7.s, #23
----------------
Maybe these should test the corner cases? If I understand the document, 14 to 28 should be legal?


================
Comment at: test/MC/AArch64/SVE/ptrues.s:6
+// RUN: llvm-mc -triple=aarch64 -filetype=obj -mattr=+sve < %s \
+// RUN:        | llvm-objdump -d -mattr=+sve - | FileCheck %s --check-prefix=CHECK-INST
+// RUN: llvm-mc -triple=aarch64 -filetype=obj -mattr=+sve < %s \
----------------
I can see a few examples of using llvm-objdump for disassembly, but why not just use llvm-mc?


https://reviews.llvm.org/D41819