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

[Joel E. Denny via Phabricator via cfe-commits]

jdenny added a comment.

In https://reviews.llvm.org/D43749#1018846, @aaron.ballman wrote:

> Committed (with whitespace fix for the test case) in r326058, thanks for the patch!


Sure.  Thanks for committing.

By the way the commit log you added isn't quite right. The issue isn't base 0 versus base 1.  The issue is attribute argument index versus attribute argument value.  See the test case for an example.

I'm not suggesting we somehow change the commit log.  I just want to be sure you see what I actually did here.


https://reviews.llvm.org/D43749