[PATCH] D60788: [Support][JSON] Add reserve() to json Array

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 16 11:30:11 PDT 2019


lebedev.ri added a comment.

As i have previously discussed in IRC with @sammccall this change needs motivational numbers.
Just copying them (+methodology) from https://reviews.llvm.org/D60609#inline-538565 sounds good.
-5.5% sounds rather great to me.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D60788/new/

https://reviews.llvm.org/D60788





More information about the llvm-commits mailing list