Index: cfe/trunk/lib/StaticAnalyzer/Checkers/MisusedMovedObjectChecker.cpp =================================================================== --- cfe/trunk/lib/StaticAnalyzer/Checkers/MisusedMovedObjectChecker.cpp +++ cfe/trunk/lib/StaticAnalyzer/Checkers/MisusedMovedObjectChecker.cpp @@ -314,17 +314,18 @@ return true; } // Function call `empty` can be skipped. - if (MethodDec && MethodDec->getDeclName().isIdentifier() && + return (MethodDec && MethodDec->getDeclName().isIdentifier() && (MethodDec->getName().lower() == "empty" || - MethodDec->getName().lower() == "isempty")) - return true; - - return false; + MethodDec->getName().lower() == "isempty")); } bool MisusedMovedObjectChecker::isStateResetMethod( const CXXMethodDecl *MethodDec) const { - if (MethodDec && MethodDec->getDeclName().isIdentifier()) { + if (!MethodDec) + return false; + if (MethodDec->hasAttr()) + return true; + if (MethodDec->getDeclName().isIdentifier()) { std::string MethodName = MethodDec->getName().lower(); if (MethodName == "reset" || MethodName == "clear" || MethodName == "destroy") @@ -429,8 +430,7 @@ // We want to investigate the whole object, not only sub-object of a parent // class in which the encountered method defined. - while (const CXXBaseObjectRegion *BR = - dyn_cast(ThisRegion)) + while (const auto *BR = dyn_cast(ThisRegion)) ThisRegion = BR->getSuperRegion(); if (isMoveSafeMethod(MethodDecl)) @@ -487,13 +487,9 @@ ThisRegion = IC->getCXXThisVal().getAsRegion(); } - for (ArrayRef::iterator I = ExplicitRegions.begin(), - E = ExplicitRegions.end(); - I != E; ++I) { - const auto *Region = *I; - if (ThisRegion != Region) { + for (const auto *Region : ExplicitRegions) { + if (ThisRegion != Region) State = removeFromState(State, Region); - } } return State; Index: cfe/trunk/test/Analysis/MisusedMovedObject.cpp =================================================================== --- cfe/trunk/test/Analysis/MisusedMovedObject.cpp +++ cfe/trunk/test/Analysis/MisusedMovedObject.cpp @@ -638,7 +638,10 @@ } } -class C : public A {}; +struct C : public A { + [[clang::reinitializes]] void reinit(); +}; + void subRegionMoveTest() { { A a; @@ -672,6 +675,13 @@ C c2 = c; // no-warning } +void resetSuperClass2() { + C c; + C c1 = std::move(c); + c.reinit(); + C c2 = c; // no-warning +} + void reportSuperClass() { C c; C c1 = std::move(c); // expected-note {{'c' became 'moved-from' here}}