[PATCH] D64833: [Xtensa 7/10] Add Xtensa instruction printer.

Andrei Safronov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 24 18:19:27 PDT 2022


andreisfr added inline comments.


================
Comment at: llvm/test/MC/Xtensa/xtensa-invalid.s:13
+# CHECK:      error: expected immediate in range [-128, 127]
+
+# imm8_sh8
----------------
saugustine wrote:
> It would be good to check instruction arguments out of order, and invalid register names.
You mean for each instruction implement extended tests with argument out of order and invalid register names?


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

https://reviews.llvm.org/D64833



More information about the llvm-commits mailing list