labath added a comment.

This is definitely a bug, but I fear that fixing this via a linear search will 
cause a big performance regression. Fortunately, since D74759 
<https://reviews.llvm.org/D74759> we now have a way to implement this more 
efficiently. Could you reimplement this function to use the binary search tree 
introduced in that patch?

If not, I can get around to it soon-ish, but I currently have a bunch of other 
things waiting in my queue...


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D77568



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
  • [Lldb-commits] [PATCH] D775... Shivam Mittal via Phabricator via lldb-commits
    • [Lldb-commits] [PATCH]... Pavel Labath via Phabricator via lldb-commits

Reply via email to