[llvm-commits] [llvm] r169626 - in /llvm/trunk: include/llvm/MC/MCAssembler.h lib/MC/MCAssembler.cpp lib/MC/MCELFStreamer.cpp lib/MC/MCMachOStreamer.cpp lib/MC/MCObjectStreamer.cpp lib/MC/MCPureStreamer.cpp lib/MC/WinCOFFStreamer.cpp
Rafael EspĂndola
rafael.espindola at gmail.com
Sat Dec 8 14:41:18 PST 2012
On 7 December 2012 14:13, Eli Bendersky <eliben at google.com> wrote:
> Author: eliben
> Date: Fri Dec 7 13:13:57 2012
> New Revision: 169626
>
> URL: http://llvm.org/viewvc/llvm-project?rev=169626&view=rev
> Log:
> Refactor MCInstFragment and MCDataFragment to adhere to a common interface,
> which removes code duplication and prepares the ground for future additions.
Thanks!
In case anyone is afraid of the performance impact (I was), I
benchmarked running llvm-mc on the -O0 clang output of gcc as a single
file (31 MB).
169625:
3.078563052 seconds time elapsed
( +- 0.42% )
169626:
3.140238572 seconds time elapsed
( +- 0.77% )
169644:
3.062390941 seconds time elapsed
( +- 0.74% )
So it looks like all the impact was from having a larger instruction
fragment, which got fixed in 169644 :-)
Cheers,
Rafael
More information about the llvm-commits
mailing list