kromanenkov added inline comments.
================ Comment at: clang/lib/StaticAnalyzer/Core/BasicValueFactory.cpp:199 - for (const auto &I : llvm::reverse(PathRange)) - PathList = prependCXXBase(I, PathList); + llvm::SmallPtrSet<QualType, 32> BaseTypes; + for (const auto &BaseSpec : PathList) ---------------- What if we get the situation of the repeated types in the `PathRange`? Like `Derived` --(cast to)-> `Base` --(cast to)-> `Derived` --(cast to)-> `Base` and so on. Do we need to store the pair of `QualType` and `PathRange` index for that case or only type will be enough? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D95307/new/ https://reviews.llvm.org/D95307 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
