rjmccall added inline comments.
================ Comment at: clang/lib/Sema/Sema.cpp:1558 + // visited before. + if (Done.count(FD)) + return; ---------------- yaxunl wrote: > rjmccall wrote: > > `insert` returns whether it changed the set, so you can combine this check > > with the insertion unless the exact ordering is important. > The order is important here. We cannot inert until the body of the function > has been traversed. Why? It only matters if we encounter `FD` again while recursing into its body, during which time `InUsePath.count(FD)` will be true and so we'll have bailed out above. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D77028/new/ https://reviews.llvm.org/D77028 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
