[PATCH] D27700: [clang-tidy] refactor ExprSequence out of misc-use-after-move check

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 11 07:17:46 PST 2017


alexfh added a comment.

One late comment.



================
Comment at: clang-tools-extra/trunk/clang-tidy/utils/ExprSequence.cpp:154-155
+const Stmt *ExprSequence::resolveSyntheticStmt(const Stmt *S) const {
+  if (SyntheticStmtSourceMap.count(S))
+    return SyntheticStmtSourceMap.lookup(S);
+  return S;
----------------
Use `.find` and store the iterator to avoid duplicate lookups.


Repository:
  rL LLVM

https://reviews.llvm.org/D27700





More information about the cfe-commits mailing list