Re: Your sim testsuite changes broke other targets

["M R Swami Reddy" <MR.Swami.Reddy@nsc.com>]

[-- Attachment #1: Type: text/plain, Size: 537 bytes --]

Hi,

Please review the attached allinsn.exp.patch file.

ChangeLog:
==
2008-04-08  M R Swami Reddy <MR.Swami.Reddy@nsc.com>

         * allinsn.exp: Removed target_alias and global_ld_options.
====

Thanks,
Swami

Daniel Jacobowitz wrote:
> On Tue, Apr 08, 2008 at 05:07:15PM +0530, M R Swami Reddy wrote:
>> +     set target_alias cr16-elf
>> +     set global_ld_options "-T$srcdir/$subdir/dummy.ld"
> 
> You shouldn't need these at all.  Why are they necessary?  The -T
> option can go in your board file if it has to go somewhere.
> 

[-- Attachment #2: allinsn.exp.patch --]
[-- Type: text/plain, Size: 594 bytes --]

Index: allinsn.exp
===================================================================
RCS file: /cvs/src/src/sim/testsuite/sim/cr16/allinsn.exp,v
retrieving revision 1.1
diff -c -r1.1 allinsn.exp
*** allinsn.exp	8 Apr 2008 09:03:17 -0000	1.1
--- allinsn.exp	8 Apr 2008 12:20:32 -0000
***************
*** 1,7 ****
  # CR16 simulator testsuite.
  
- set target_alias cr16-elf
- set global_ld_options "-L/local/user/swami/cr16-5.1/build/CR_R_2007-08-24_11-34-56/release/linux/lib/gcc/cr16-elf/4.1.2 -T/home10/swami/dummy.ld"
  
  if [istarget cr16*-*-*] {
      # load support procs
--- 1,5 ----