[PATCH] D104844: [Analyzer][solver] Fix crashes during symbol simplification

Valeriy Savchenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jun 25 02:56:28 PDT 2021


vsavchenko added inline comments.


================
Comment at: clang/lib/StaticAnalyzer/Core/RangeConstraintManager.cpp:1981
   for (const SymbolRef &MemberSym : ClassMembers) {
-    SymbolRef SimplifiedMemberSym = ::simplify(State, MemberSym);
+    SymbolRef SimplifiedMemberSym = ::clang::ento::simplify(State, MemberSym);
     if (SimplifiedMemberSym && MemberSym != SimplifiedMemberSym) {
----------------
martong wrote:
> vsavchenko wrote:
> > Oof, and there is no way to avoid all these namespaces?
> Unfortunately, the candidate function would be the member `simplify` in that case. Though, we can get rid of the `::clang` prefix.
OK, at least `ento::`!


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D104844/new/

https://reviews.llvm.org/D104844



More information about the cfe-commits mailing list