[llvm] r212837 - Simplify the raw_svector_ostream tweak from r212816
chandlerc at google.com
Fri Jul 11 11:47:09 PDT 2014
On Fri, Jul 11, 2014 at 11:23 AM, Alp Toker <alp at nuanti.com> wrote:
> Simplify the raw_svector_ostream tweak from r212816
> The memcpy() and overlap helps didn't help much with timings, so clean up
> the change.
> The difference at this point is that we now leave growth of the storage
> up to SmallVector's implementation:
> - OS.reserve(OS.capacity() * 2);
> + OS.reserve(OS.size() + 64);
Thanks for fixing this, but in the future, it would be a lot easier to
review this kind of patch by reverting the original and then re-applying
just the cleanup portion of the patch.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits