[Lldb-commits] [PATCH] D149949: [lldb][TypeSystem] ForEach: Don't hold the TypeSystemMap lock across callback

[Michael Buch via Phabricator via lldb-commits]

Michael137 added a comment.

In D149949#4341608 <https://reviews.llvm.org/D149949#4341608>, @labath wrote:

> Just as a drive-by, since we're constructing a copy anyway it would be possible to merge this construction with the "uniqueing" behavior of the `visited` set. I.e., instead of making a literal copy of the map, just construct a `to_be_visited` set of /unique/ objects, and then iterate over that.

Agreed that would be a nice cleanup. I considered it initially but thought I'd split it out into a separate patch (but never did)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D149949/new/

https://reviews.llvm.org/D149949