Skip to content

Commit 97187e1

Browse files
[AArch64] update "rm" inline asm test (#67472)
Because `x0` is not listed in the clobber list, regalloc could (one day when #20571 is fixed) allocate `$0` to `x0`: ldr x0, x0 This will produce an error when validating the instruction. The intent of this test FWICT is to check that the parameter in w0 is stored to a stack slot using w0, since this target triple is the exotic arm64_32 (ILP32). Update the test to simply use "m" constraint. The clobber list is underconstrained otherwise.
1 parent 4bdec58 commit 97187e1

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

llvm/test/CodeGen/AArch64/arm64_32.ll

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -649,7 +649,7 @@ define <2 x ptr> @test_pointer_vec_load(ptr %addr) {
649649
define void @test_inline_asm_mem_pointer(ptr %in) {
650650
; CHECK-LABEL: test_inline_asm_mem_pointer:
651651
; CHECK: str w0,
652-
tail call void asm sideeffect "ldr x0, $0", "rm"(ptr %in)
652+
tail call void asm sideeffect "ldr x0, $0", "m"(ptr %in)
653653
ret void
654654
}
655655

0 commit comments

Comments
 (0)