sdesmalen added inline comments.

================
Comment at: clang/test/CodeGen/aarch64-sve-inline-asm.c:3
+// RUN:   -emit-llvm -o - %s | FileCheck %s -check-prefix=CHECK
 
 void test_sve_asm(void) {
----------------
Could you also add a RUN line that compiles this to asm, just to make sure that 
LLVM accepts the inline asm syntax? There don't need to be any checks for the 
output.
This then probably also requires a `REQUIRES: aarch64-registered-target`.


================
Comment at: clang/test/CodeGen/aarch64-sve-inline-asm.c:18
+void test_sve2p1_asm(void) {
+  register __SVCount_t x2 asm("pn0");
+  asm("ptrue pn8.b" ::: "pn8");
----------------
x2 seems unused. Did you forget to assign it and to add some CHECK lines for it?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156115

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to