[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 cfe-commits at lists.llvm.org
Mon Jul 2 11:59:09 PDT 2018


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





More information about the cfe-commits mailing list