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

Gabor Marton via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 2 09:55:33 PST 2020


martong added inline comments.


================
Comment at: clang/include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h:932
+  const CXXConstructExpr *getInheritingConstructor() const {
+    return cast<CXXConstructExpr>(getInheritingStackFrame()->getCallSite());
+  }
----------------
This line causes a regression in our CTU jobs, because it seems `getCallSite` can return with a null Stmt*. And then cast fires an assertion. I am going to do a simple fix by changing `cast` to `cast_or_null` if there is no objection.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74735





More information about the cfe-commits mailing list