[PATCH] D12759: [clang-tidy] Add misc-sizeof-container check to find sizeof() uses on stlcontainers.

Kim Gräsman via cfe-commits cfe-commits at lists.llvm.org
Sun Sep 13 01:52:43 PDT 2015


Late to the party, but I wanted to ask: is there a way to indicate to
the checker that we really *did* mean sizeof()?

I think I've stumbled over code in our code base that uses
sizeof(container) to report memory usage statistics and it seems
valid, so it'd be nice if this checker could be silenced on a
case-by-case basis.

Thanks,
- Kim

On Sat, Sep 12, 2015 at 12:09 AM, Alexander Kornienko via cfe-commits
<cfe-commits at lists.llvm.org> wrote:
> Indeed. But this has been fixed before I could get to it.
>
>
> On Thu, Sep 10, 2015 at 10:47 PM, Aaron Ballman via cfe-commits
> <cfe-commits at lists.llvm.org> wrote:
>>
>> aaron.ballman added a comment.
>>
>> This appears to have broken one of the bots:
>>
>> http://bb.pgr.jp/builders/ninja-x64-msvc-RA-centos6/builds/15065
>>
>>
>> http://reviews.llvm.org/D12759
>>
>>
>>
>> _______________________________________________
>> cfe-commits mailing list
>> cfe-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>


More information about the cfe-commits mailing list