[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:03:16 PDT 2018


sbaranga added a comment.

IIRC the contents of the asm string shouldn't matter here, and we would expect to get an error in the assembler instead when trying to assemble the output (which is fine, since this is a misuse of the X constraint).

However, it seems wrong for the compiler to assign s0 as an operand without having FP registers available.


https://reviews.llvm.org/D53748





More information about the llvm-commits mailing list