Re: scan-assembler harness bug?

2008-10-24 Thread DJ Delorie
Committed as attached. Thanks! > Sure. 2008-10-24 DJ Delorie <[EMAIL PROTECTED]> * lib/scanasm.exp (scan-assembler, scan-assembler-not, scan-hidden, scan-not-hidden, scan-file, scan-file-not, scan-assembler-times, scan-assembler-dem, scan-assembler-dem-not): E

Re: scan-assembler harness bug?

2008-10-24 Thread Janis Johnson
On Thu, 2008-10-23 at 22:01 -0400, DJ Delorie wrote: > > OK, after you've run the testsuite with this change. The ChangeLog > > entry should show the names of all of the procs you changed. > > Is gcc.target/i386.exp enough? I originally found it with > xstormy16-elf on an older branch, but xstor

Re: scan-assembler harness bug?

2008-10-23 Thread DJ Delorie
> OK, after you've run the testsuite with this change. The ChangeLog > entry should show the names of all of the procs you changed. Is gcc.target/i386.exp enough? I originally found it with xstormy16-elf on an older branch, but xstormy16 isn't a happy target in trunk at the moment.

Re: scan-assembler harness bug?

2008-10-23 Thread Janis Johnson
On Thu, 2008-10-23 at 19:12 -0400, DJ Delorie wrote: > > I thought something like that was already done but I couldn't find it > > right now. Is it a problem? If so, we should certainly do this. > > I see it done in other places, but not scanasm. How's this? > > 2008-10-23 DJ Delorie <[EMAIL

Re: scan-assembler harness bug?

2008-10-23 Thread DJ Delorie
> I thought something like that was already done but I couldn't find it > right now. Is it a problem? If so, we should certainly do this. I see it done in other places, but not scanasm. How's this? 2008-10-23 DJ Delorie <[EMAIL PROTECTED]> * lib/scanasm.exp: Extract first word of $

Re: scan-assembler harness bug?

2008-10-23 Thread Janis Johnson
On Thu, 2008-10-23 at 18:00 -0400, DJ Delorie wrote: > In scanasm.exp we do this: > > proc scan-assembler { args } { > upvar 2 name testcase > set output_file "[file rootname [file tail $testcase]].s" > > > However, in dejagnu, $name may have compiler switches appended to it: > > #

scan-assembler harness bug?

2008-10-23 Thread DJ Delorie
In scanasm.exp we do this: proc scan-assembler { args } { upvar 2 name testcase set output_file "[file rootname [file tail $testcase]].s" However, in dejagnu, $name may have compiler switches appended to it: # We append the compilation flags, if any, to ensure that the test case