[PATCH] D53748: [ARM] Fix test inlineasm-X-allocation.ll

silviu.baranga@arm.com via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 26 04:51:23 PDT 2018


sbaranga added a comment.

The initial intent was to make sure that we are forcing the operands to not be VFP registers, so I would have expected to get a vadd.f32 r0, r0, r0. However, since the instruction is invalid it doesn't make a good test (as you observed).

Using a

call void asm sideeffect "// foo $0, $0, $0", "X" (float %f) nounwind

instruction would work just as well for this test and it wouldn't produce an assembler error.


https://reviews.llvm.org/D53748





More information about the llvm-commits mailing list