[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