martong marked 2 inline comments as done.
martong added a comment.
Valeriy, thanks for the assiduous and quick review!
================
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) {
----------------
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.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D104844/new/
https://reviews.llvm.org/D104844
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
