[Lldb-commits] [PATCH] D23406: Fix a race in Broadcaster/Listener interaction

Enrico Granata via lldb-commits lldb-commits at lists.llvm.org
Thu Aug 11 16:00:59 PDT 2016


> On Aug 11, 2016, at 3:55 PM, Jim Ingham via lldb-commits <lldb-commits at lists.llvm.org> wrote:
> 
> jingham added a comment.
> 
> The patch seems correct to me.
> 
> I don't have a strong opinion about std::vector vrs. SmallVector.  These are temporary objects, so the size of the container doesn't matter, and I doubt this code is hot enough in normal lldb sessions that the difference between in performance between the two will matter.  Maybe the SmallVector data formatter (llvm/utils/lldbDataFormatters.py) works, or if it doesn't we should fix it?

IIRC, Argyrios wrote those formatters a few years ago, so +Argyrios to comment on whether it's expected to work

> 
> 
> https://reviews.llvm.org/D23406
> 
> 
> 
> _______________________________________________
> lldb-commits mailing list
> lldb-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits


Thanks,
- Enrico
📩 egranata@.com ☎️ 27683

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/lldb-commits/attachments/20160811/c80f3332/attachment.html>


More information about the lldb-commits mailing list