[PATCH] D55387: [analyzer] MoveChecker Pt.7: NFC: Misc refactoring.

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 6 14:59:36 PST 2018


NoQ created this revision.
NoQ added reviewers: dcoughlin, xazax.hun, a_sidorin, george.karpenkov, szepet, rnkovacs, Szelethus.
Herald added subscribers: cfe-commits, dkrupp, donat.nagy, mikhail.ramalho, a.sidorin, baloghadamsoftware.
NoQ added a parent revision: D55307: [analyzer] MoveChecker Pt.6: Suppress the warning for the few move-safe STL classes..

This is a no-functional-change part of the next patch.

The checker supports 5 different sorts of "use" after move: copying, moving, copy-assigning from, move-assigning from, calling any other method. De-duplicate the common checks that repeat 5 times into a single method, `checkUse()`.

Re-structure the code a little bit.


Repository:
  rC Clang

https://reviews.llvm.org/D55387

Files:
  lib/StaticAnalyzer/Checkers/MoveChecker.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D55387.177050.patch
Type: text/x-patch
Size: 10335 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181206/d24c93a3/attachment.bin>


More information about the cfe-commits mailing list