jasonmolenda added a comment.

This all looks good to me, thanks for doing this Pavel.  Tamas asked in an 
earlier comment, "The previous version of the data formatter was triggering for 
std::vector<std::allocator<bool>> as well. Jason, do you know why was it the 
case? Do we need that functionality because of a broken compiler version or can 
it be removed?"

I have no idea why that was there.  I could dig around in the svn commit 
history to try to figure it out, but it may have been unnecessary to begin 
with.  If in doubt, we can drop it and put it back in if we hear of any 
problems.


https://reviews.llvm.org/D31880



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

Reply via email to