[PATCH] D48617: [Builtins][Attributes][X86] Tag all X86 builtins with their required vector width. Add a min_vector_width function attribute and tag all x86 instrinsics with it.

[Craig Topper via Phabricator via cfe-commits]

craig.topper added inline comments.


================
Comment at: test/Sema/attr-min-vector-width.c:8
+
+void f3(void) __attribute__((__min_vector_width__(128), 
__min_vector_width__(256))); /* expected-warning {{attribute 
'__min_vector_width__' is already applied with different parameters}} */
----------------
aaron.ballman wrote:
> Also missing tests for applying the attribute to the wrong subject and with 
> the incorrect number of arguments.
How many possible subjects are there? And how many wrong subjects do you want 
me to test?


https://reviews.llvm.org/D48617



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