JDevlieghere accepted this revision. JDevlieghere added a comment. In D122980#3426196 <https://reviews.llvm.org/D122980#3426196>, @labath wrote:
> In D122980#3424636 <https://reviews.llvm.org/D122980#3424636>, @JDevlieghere > wrote: > >> 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. > > This is pretty new code, so I think you must be thinking of something else. > > I don't see any reason why this would have to be a std::map. I think the code > just hasn't been optimized yet. Cool, thanks for confirming. LGTM! 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
