[PATCH] D44231: [clang-tidy] Check for sizeof that call functions

Paul Fultz II via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 16 15:47:15 PDT 2018


pfultz2 added a comment.

> As this patch can catch some mistakes, I'm fine with checking it in. I agree that it is reasonable to write normal code like sizeof(func_call()) (not false positive), maybe set the option to false by default?

I have disabled it by default. We can decide later if we want it to enabled by default.


https://reviews.llvm.org/D44231





More information about the cfe-commits mailing list