[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