clayborg added a comment.

My only issue with this patch is that is uses llvm::SmallVector. Although the 
class is nice, we don't have visibility into this class when debugging like we 
do when we use STL collections. I would rather not move to SmallVector if we 
don't need to. Jim, thoughts?


https://reviews.llvm.org/D23406



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to