yaxunl marked an inline comment as done.
yaxunl added inline comments.

================
Comment at: test/Sema/inline-asm-validate-amdgpu.cl:74
+    __asm("v_add_f64_e64 v[1:2], v[3:4], v[5:6]" : "=v[1:2]"(ci) : 
"v[3:4]"(ai), "v[5:6]"(bi) : ); //expected-error {{invalid output constraint 
'=v[1:2]' in asm}}
+
+    c[i] = ci;
----------------
yaxunl wrote:
> arsenm wrote:
> > While you're here can we add some tests for the immediate constraints? 
> > There was a bug report recently when using s_trap with the i constraints 
> > for the constant operand.
> what's the syntax of the immediate constraints? And some examples? Thanks.
Ping. Can we leave this for another patch? Since there are people waiting for 
this feature.


https://reviews.llvm.org/D37568



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to