[PATCH] D77028: Speed up deferred diagnostic emitter

[John McCall via Phabricator via cfe-commits]

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