JDevlieghere added a comment.

I feel like this came up in the past and there was a reason an unordered map 
couldn't work? Maybe I'm confusing this with something else. Added Pavel as he 
would certainly know.

Could we use a `llvm::DenseMap` here, as it's supposedly even faster than its 
STL counterpart?

(Adding myself as a reviewer too so this shows up in my review queue)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122980

_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
  • [Lldb-commits] [PATCH] ... Luboš Luňák via Phabricator via lldb-commits
    • [Lldb-commits] [PA... Jonas Devlieghere via Phabricator via lldb-commits

Reply via email to