Charusso marked 3 inline comments as done. Charusso added a comment. Sorry, I was sure it is working as expected.
================ Comment at: clang/lib/StaticAnalyzer/Core/DynamicType.cpp:53-57 if (const auto *TR = dyn_cast<TypedRegion>(MR)) - return DynamicTypeInfo(TR->getLocationType(), /*CanBeSub=*/false); + return new DynamicTypeInfo(TR->getLocationType(), /*CanBeSub=*/false); - if (const auto *SR = dyn_cast<SymbolicRegion>(MR)) { - SymbolRef Sym = SR->getSymbol(); - return DynamicTypeInfo(Sym->getType()); - } + if (const auto *SR = dyn_cast<SymbolicRegion>(MR)) + return new DynamicTypeInfo(SR->getSymbol()->getType()); ---------------- NoQ wrote: > Do i have to do a `delete` manually every time i call this function? Well, from now you do not. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D68199/new/ https://reviews.llvm.org/D68199 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits