[Lldb-commits] [PATCH] D134906: Have MemoryCache cache addresses that were unreadable, so duplicate read attempts can be suppressed
Jim Ingham via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Fri Sep 30 11:00:01 PDT 2022
jingham added a comment.
LGTM
I didn't see a more appropriate datatype than SmallSet in the llvm collection.
I wondered about the same thing Dave asked - should the errors mention that this failed because we checked a negative cache - but I think that would be more confusing than helpful to lldb users. If we wanted to track these decisions it would be more appropriate to log them, but I'm not sure even that is necessary.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D134906/new/
https://reviews.llvm.org/D134906
More information about the lldb-commits
mailing list