Manna 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);
----------------
aaronpuchert wrote:
> Can't we just take `I.getData()` instead of doing a lookup? After all we're
> iterating over the same map.
Thank you @aaronpuchert for review and comment.
>>Can't we just take I.getData() instead of doing a lookup? After all we're
>>iterating over the same map.
Makes sense to me! I have updated patch
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D153033/new/
https://reviews.llvm.org/D153033
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
