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

Aaron Ballman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sun Feb 25 11:57:51 PST 2018


aaron.ballman added a comment.

In https://reviews.llvm.org/D43749#1018825, @jdenny wrote:

> In https://reviews.llvm.org/D43749#1018818, @aaron.ballman wrote:
>
> > Aside from a minor testcase nit, this LGTM. Why is this dependent on https://reviews.llvm.org/D43248?
>
>
> The dependency goes the other way.  Did I get it wrong?


Ugh, no, I misunderstood what Phab was trying to tell me. This makes considerably more sense. :-)


https://reviews.llvm.org/D43749





More information about the cfe-commits mailing list