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

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 12 22:22:14 PDT 2022


MaskRay added inline comments.


================
Comment at: llvm/test/MC/Xtensa/elf-header.s:5
+# Xtensa: Format: elf32-xtensa
+# Xtensa: Arch: xtensa
+# Xtensa: AddressSize: 32bit
----------------
Add `-NEXT` when applicable.


================
Comment at: llvm/test/MC/Xtensa/xtensa-valid.s:1
+# RUN: llvm-mc %s -triple=xtensa -show-encoding \
+# RUN:     | FileCheck -check-prefixes=CHECK,CHECK-INST %s
----------------
Split this into multiple files.


MC/AArch64/armv8* and MC/LoongArch/Basic can give some insights how tests are organized.


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

https://reviews.llvm.org/D64833



More information about the llvm-commits mailing list