[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]

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