[PATCH] D43749: [Attr] Fix alloc_size's diags to report arg idx not value

[Aaron Ballman via Phabricator via cfe-commits]

aaron.ballman accepted this revision.
aaron.ballman added a comment.
This revision is now accepted and ready to land.

Aside from a minor testcase nit, this LGTM. Why is this dependent on https://reviews.llvm.org/D43248?



================
Comment at: test/Sema/alloc-size.c:3
 
-void *fail1(int a) __attribute__((alloc_size)); //expected-error{{'alloc_size' attribute takes at least 1 argument}}
+void *fail1(int a) __attribute__((alloc_size));   //expected-error{{'alloc_size' attribute takes at least 1 argument}}
 void *fail2(int a) __attribute__((alloc_size())); //expected-error{{'alloc_size' attribute takes at least 1 argument}}
----------------
This looks like a whitespace-only change?


https://reviews.llvm.org/D43749