[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