dcoughlin added a comment.

A 6% speed improvement could be a big win! Do we have a sense of what the 
expected increased memory cost (as a % of average use over the lifetime of the 
process) is? My guess is it would be relatively low. I suspect most analyzer 
users run relatively few concurrent 'clang' processes -- so this might be well 
worth it.

I do think we should make sure the user can't shoot themselves in the foot by 
pre-reserving space for an absurdly high maximum step count. We might want to 
to clamp this reservation to something that is not outrageous even when the 
maximum step count is huge.


http://reviews.llvm.org/D20933



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

Reply via email to