[RFA]: testsuite/gdb.asm/asm-sources.exp: Allow defining linker flags

[RFA]: testsuite/gdb.asm/asm-sources.exp: Allow defining linker flags

[Corinna Vinschen]

Hi,

the following patch allows to have linker flags in gdb.asm/asm-sources.exp:

2001-11-13  Corinna Vinschen  <vinschen@redhat.com>

	* gdb.asm/asm-sources.exp: Allow defining linker flags.

Index: gdb.asm/asm-source.exp
===================================================================
RCS file: /cvs/src/src/gdb/testsuite/gdb.asm/asm-source.exp,v
retrieving revision 1.5
diff -u -p -r1.5 asm-source.exp
--- asm-source.exp	2001/11/10 01:44:43	1.5
+++ asm-source.exp	2001/11/13 22:30:19
@@ -33,6 +33,7 @@ set bug_id 0
 
 set asm-arch ""
 set asm-flags ""
+set link-flags ""
 
 if [istarget "d10v-*-*"] then {
     set asm-arch d10v
@@ -58,7 +59,7 @@ if {[target_assemble ${src2} asmsrc2.o "
      gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
 }
 
-if  { [target_link "asmsrc1.o asmsrc2.o" ${binfile} ""] != "" } {
+if  { [target_link "asmsrc1.o asmsrc2.o" ${binfile} "${link-flags}"] != "" } {
      gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
 }
 
-- 
Corinna Vinschen
Cygwin Developer
Red Hat, Inc.
mailto:vinschen@redhat.com

Re: [RFA]: testsuite/gdb.asm/asm-sources.exp: Allow defining linker flags

[Fernando Nasser]

Corinna Vinschen wrote:
> 
> Hi,
> 
> the following patch allows to have linker flags in gdb.asm/asm-sources.exp:
> 

If you need it go ahead.  It shouldn't hurt anyone.

Fernando


> 2001-11-13  Corinna Vinschen  <vinschen@redhat.com>
> 
>         * gdb.asm/asm-sources.exp: Allow defining linker flags.
> 
> Index: gdb.asm/asm-source.exp
> ===================================================================
> RCS file: /cvs/src/src/gdb/testsuite/gdb.asm/asm-source.exp,v
> retrieving revision 1.5
> diff -u -p -r1.5 asm-source.exp
> --- asm-source.exp      2001/11/10 01:44:43     1.5
> +++ asm-source.exp      2001/11/13 22:30:19
> @@ -33,6 +33,7 @@ set bug_id 0
> 
>  set asm-arch ""
>  set asm-flags ""
> +set link-flags ""
> 
>  if [istarget "d10v-*-*"] then {
>      set asm-arch d10v
> @@ -58,7 +59,7 @@ if {[target_assemble ${src2} asmsrc2.o "
>       gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
>  }
> 
> -if  { [target_link "asmsrc1.o asmsrc2.o" ${binfile} ""] != "" } {
> +if  { [target_link "asmsrc1.o asmsrc2.o" ${binfile} "${link-flags}"] != "" } {
>       gdb_suppress_entire_file "Testcase compile failed, so all tests in this file will automatically fail."
>  }
> 
> --
> Corinna Vinschen
> Cygwin Developer
> Red Hat, Inc.
> mailto:vinschen@redhat.com

-- 
Fernando Nasser
Red Hat - Toronto                       E-Mail:  fnasser@redhat.com
2323 Yonge Street, Suite #300
Toronto, Ontario   M4P 2C9

Re: [RFA]: testsuite/gdb.asm/asm-sources.exp: Allow defining linker flags

[Corinna Vinschen]

On Wed, Nov 14, 2001 at 10:44:49AM -0500, Fernando Nasser wrote:
> Corinna Vinschen wrote:
> > 
> > Hi,
> > 
> > the following patch allows to have linker flags in gdb.asm/asm-sources.exp:
> > 
> 
> If you need it go ahead.  It shouldn't hurt anyone.
> 
> Fernando

Thanks. Applied.

Corinna