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

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 16 04:24:50 PDT 2018


hokein 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?


https://reviews.llvm.org/D44231





More information about the cfe-commits mailing list