NoQ added inline comments.
================
Comment at: clang/lib/StaticAnalyzer/Core/ExprEngineCXX.cpp:619
getCheckerManager().runCheckersForEvalCall(DstEvaluated, *I, *Call,
*this,
- CallOpts);
+ CallOpts, Bldr);
}
----------------
vrnithinkumar wrote:
> NoQ wrote:
> > We should probably delete the copy-constructor for node builders. I've no
> > idea what it's supposed to do anyway and the whole problem that we're
> > having here is due to there being //too many// of them already.
> So we should disable the copying of `NodeBuilder` and create a heap allocated
> `NodeBuilder` and use pointer to pass around functions?
No-no, keep it on the stack and don't pass it around *at all*.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85796/new/
https://reviews.llvm.org/D85796
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
