[PATCH] D22834: Added 'inline' attribute to basic_string's destructor
Eric Fiselier via cfe-commits
cfe-commits at lists.llvm.org
Tue Aug 2 22:48:28 PDT 2016
EricWF added a subscriber: EricWF.
EricWF added a comment.
LGTM.
However I would like to see a small benchmark that demonstrates the performance change. Please try and write the benchmark using Google Benchmark.
Some helpful links:
http://libcxx.llvm.org/docs/TestingLibcxx.html#building-benchmarks
http://github.com/google/benchmark
Let me know if I can help.
https://reviews.llvm.org/D22834
More information about the cfe-commits
mailing list