steakhal added inline comments. Herald added a subscriber: ASDenysPetrov.
================ Comment at: clang/include/clang/StaticAnalyzer/Core/PathSensitive/CallEvent.h:916-918 + virtual const CXXInheritedCtorInitExpr *getOriginExpr() const { + return cast<CXXInheritedCtorInitExpr>(AnyFunctionCall::getOriginExpr()); + } ---------------- Why is this function virtual? If we want such behavior we should mark the `CallEvent::getOriginExpr` virtual and just //override// it here. As-of-now, this just hides the previous implementation, causing potential problems. This code-smell occures several times across this class hierachy. Is this the expected behavior @NoQ? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D74735/new/ https://reviews.llvm.org/D74735 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits