[PATCH] D19061: [ARM] Add support for the X asm constraint
silviu.baranga@arm.com via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 25 03:38:12 PDT 2016
sbaranga added inline comments.
================
Comment at: test/CodeGen/ARM/inlineasm-X-constraint.ll:4
@@ +3,3 @@
+; CHECK-LABEL: f1
+; CHECK: vmsr fpscr
+; CHECK: vadd.f64
----------------
rengolin wrote:
> shouldn't you verify that the register used in vmsr is also used in vadd?
>
> Same comment for other CHECK lines below.
The vadd operand ($0) doesn't appear in the vmsr instruction. It's there just to add the dependency (which is the usecase shown in the PR).
http://reviews.llvm.org/D19061
More information about the llvm-commits
mailing list