This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PATCH: Add a testcase for PR gas/14201


On Thu, Jun 07, 2012 at 07:43:25AM -0700, H.J. Lu wrote:
> On Thu, Jun 7, 2012 at 7:22 AM, Alan Modra <amodra@gmail.com> wrote:
> > On Thu, Jun 07, 2012 at 06:48:28AM -0700, H.J. Lu wrote:
> >> I checked in this patch to add a testcase for PR gas/14201.
> >
> > Please revert. ?The testcase takes too long to run.
> >
> 
> I timed it on a 7 years old 32-bit dual-core machine.  The total
> times of "make check -j4" are

make -j hides the real impact, and the testcase will likely fail on
machines without a large amount of memory.  

> It is 1 more second.  I will move it to i386 directory.

My changes to sb.c were really about removing an arbitrary limit on
string buffer size.  It's not something that we need to test for,
surely!  The test in the PR allocated a 1G string buffer.  I see now
that your test is "only" 500M, but even so that is a ridiculous amount
of memory to require for an assembler test.

-- 
Alan Modra
Australia Development Lab, IBM


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]