[PATCH] D64969: [llvm-objdump][NFC] Make the PrettyPrinter::printInst() output buffered

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 23 23:09:30 PDT 2019


MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.

Apologize I added formatted_raw_ostream and I didn't read its ctor carefully...

> Benchmark #2: ./bin/llvm-objdump -d ./bin/llvm-objcopy
> 
>   Time (mean ± σ):     386.4 ms ±  13.0 ms    [User: 376.6 ms, System: 6.1 ms]
>   Range (min … max):   366.1 ms … 407.0 ms    10 runs

Please remember updating the description before you commit. The benchmark has been updated.


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

https://reviews.llvm.org/D64969





More information about the llvm-commits mailing list