veluca93 marked an inline comment as done.
veluca93 added inline comments.
================
Comment at:
clang-tools-extra/clang-tidy/performance/UnusedNoSideEffectCheck.cpp:135
+ // or class, and that there are no side effects on `this`.
+ if (auto *Callee = dyn_cast<MemberExpr>(Call->getCallee())) {
+ markSideEffectFree(Callee);
----------------
Eugene.Zelenko wrote:
> `const auto *`. Same in other similar places.
Somehow I missed a couple of those... should be fixed now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D124918/new/
https://reviews.llvm.org/D124918
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
