[PATCH] D124918: [clang-tidy] Add a new check for non-trivial unused variables.

[Eugene Zelenko via Phabricator via cfe-commits]

Eugene.Zelenko 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);
----------------
`const auto *`. Same in other similar places.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D124918/new/

https://reviews.llvm.org/D124918

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits