[PATCH] D43659: [analyzer] Don't crash when dynamic type of a concrete region is hard-set with placement new.

Artem Dergachev via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 22 17:24:31 PST 2018


NoQ added inline comments.


================
Comment at: lib/StaticAnalyzer/Core/CallEvent.cpp:593
+        // Fall back to a generic pointer cast for this-value.
+        const CXXMethodDecl *StaticMD = cast<CXXMethodDecl>(getDecl());
+        const CXXRecordDecl *StaticClass = StaticMD->getParent();
----------------
george.karpenkov wrote:
> nit: auto?
Dunno, i somehow like the symmetry with the next line. "Take method decl, take record decl". Maybe it's just me.


https://reviews.llvm.org/D43659





More information about the cfe-commits mailing list