[PATCH] D154899: [SystemZ] Allow symbols in immediate asm operands
Ilya Leoshkevich via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Jul 20 12:39:31 PDT 2023
iii added inline comments.
================
Comment at: llvm/test/MC/SystemZ/fixups.s:8
+# RUN: llvm-mc -triple s390x-unknown-unknown -mcpu=z13 -filetype=obj %s | \
+# RUN: llvm-objdump -d - | FileCheck %s -check-prefix=CHECK-DIS
+
----------------
uweigand wrote:
> Would it make sense to add `CHECK-DIS` lines to the existing tests as well?
For the new tests it's important, because it allows us to check the resolved immediate value. The existing tests depend relocations, so it's going to be only zeroes anyway.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154899/new/
https://reviews.llvm.org/D154899
More information about the llvm-commits
mailing list