MaskRay added a comment.

> Context not available.

See `-U999999` on 
https://llvm.org/docs/Phabricator.html#requesting-a-review-via-the-web-interface
 or use `arc diff` (PHP 8 may need 
https://discourse.llvm.org/t/arcanist-deprecation-errors-on-php-8/63231)



================
Comment at: llvm/test/MC/RISCV/zawrs-valid.s:2
+# RUN: llvm-mc %s -triple=riscv32 -mattr=+zawrs \
+# RUN:     | FileCheck -check-prefixes=CHECK-S-OBJ %s
+# RUN: llvm-mc %s -triple=riscv64 -mattr=+zawrs \
----------------
The convention is to use `--check-prefix` is there is only one prefix.


================
Comment at: llvm/test/MC/RISCV/zawrs-valid.s:12
+
+# CHECK-S-OBJ: wrs
+wrs
----------------
Append `{{$}}` to make the test stricter.


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

https://reviews.llvm.org/D128235

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

Reply via email to