[llvm-commits] [llvm] r164724 - in /llvm/trunk: lib/Target/X86/MCTargetDesc/X86MachObjectWriter.cpp test/MC/MachO/i386-large-relocations.s

[Benjamin Kramer]

On 01.10.2012, at 03:03, Chandler Carruth <chandlerc at google.com> wrote:

> On Wed, Sep 26, 2012 at 2:27 PM, Jim Grosbach <grosbach at apple.com> wrote:
> Author: grosbach
> Date: Wed Sep 26 16:27:45 2012
> New Revision: 164724
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=164724&view=rev
> Log:
> X86_32: Large Symbol+Offset relocations.
> 
> If the offset is more than 24-bits, it won't fit in a scattered
> relocation offset field, so we fall back to using a non-scattered
> relocation.
> 
> rdar://12358909
> 
> Added:
>     llvm/trunk/test/MC/MachO/i386-large-relocations.s
> 
> Jim, it looks like this is the slowest running test in the entire regression test suite now... it runs for over 30s on my system, where the total lit runtime is 52 seconds =/
> 
> Is there any way to prune it down to a smaller test? Or maybe there is some N^2 aspect to this?
> 
> The time appears to go to llvm-mc rather than macho-dump, but I've not really dug into why. I just wanted to let you know.

It just used an extremely complicated way to emit a stream of zero bytes. r164945 brings it down from 27s to 0.6s in a Debug+Asserts build.

- Ben