aaronpuchert added inline comments.
================
Comment at: clang/lib/Analysis/ThreadSafety.cpp:505-511
const unsigned *i = C.lookup(D);
llvm::errs() << " -> ";
+ if (!i) {
+ llvm::errs() << "<<NULL>>";
+ return;
+ }
dumpVarDefinitionName(*i);
----------------
Can't we just take `I.getData()` instead of doing a lookup? After all we're
iterating over the same map.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153033/new/
https://reviews.llvm.org/D153033
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
