yaxunl added inline comments.
================
Comment at: test/Sema/inline-asm-validate-amdgpu.cl:38
+ __asm("v_add_f32_e32 v1, v2, v3" : "=v1"(ci) : "v2"(ai), "v3"(bi) : ); ///
expected-error {{invalid output constraint '=v1' in asm}}
+ __asm("v_add_f32_e32 v1, v2, v3" : "=v1:2"(ci) : "v3:4"(ai), "v4:5"(bi) :
); // expected-error {{invalid output constraint '=v1:2' in asm}}
+
----------------
arsenm wrote:
> arsenm wrote:
> > I don't understand. The backend parsed syntax is v[3:4]. Why should v3:4 be
> > valid? Also in this example you are using a 64-bit input to a 32-bit operand
> i.e. this won't parse in the backend and is invalid. The brackets are required
Will fix the syntax about [].
The test did not consider validity of the inline assembly for the backend since
FE only check format. I agree it is better to make it valid for backend and
will try to fix that.
https://reviews.llvm.org/D37568
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
