[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