[PATCH] D74735: [analyzer] Add support for CXXInheritedCtorInitExpr.

Gábor Horváth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Feb 18 04:29:27 PST 2020


xazax.hun added a comment.

If the AST is hard to work with would it make sense to try to change the AST a bit?



================
Comment at: clang/lib/StaticAnalyzer/Core/CallEvent.cpp:944
+  if (Data) {
+    loc::MemRegionVal MV(static_cast<const MemRegion *>(Data));
+    if (SymbolRef Sym = MV.getAsSymbol(true))
----------------
Maybe reusing `getCXXThisVal` here to reduce the number of casts?


Repository:
  rC Clang

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

https://reviews.llvm.org/D74735





More information about the cfe-commits mailing list