[PATCH] testsuite: Fix gcc.target/arm/multilib.exp use of gcc_opts

Richard Sandiford richard.sandiford@arm.com
Tue Aug 11 16:40:24 GMT 2020


Christophe Lyon via Gcc-patches <gcc-patches@gcc.gnu.org> writes:
> This patch fixes an incorrect parameter passing for $gcc_opts, which
> produces a DejaGnu error: (DejaGnu) proc "gcc_opts" does not exist.

Huh, wonder how that went unnoticed for so long…

> 2020-08-11  Christophe Lyon  <christophe.lyon@linaro.org>
>
>         gcc/testsuite/
>         * gcc.target/arm/multilib.exp: Fix parameter passing for gcc_opts.

OK everywhere that needs it, thanks.

Richard

> diff --git a/gcc/testsuite/gcc.target/arm/multilib.exp b/gcc/testsuite/gcc.target/arm/multilib.exp
> index f67a92a..c5f3c02 100644
> --- a/gcc/testsuite/gcc.target/arm/multilib.exp
> +++ b/gcc/testsuite/gcc.target/arm/multilib.exp
> @@ -40,7 +40,7 @@ proc multilib_config {profile} {
>  proc check_multi_dir { gcc_opts multi_dir } {
>      global tool
>  
> -    set options [list "additional_flags=[concat "--print-multi-directory" [gcc_opts]]"]
> +    set options [list "additional_flags=[concat "--print-multi-directory" $gcc_opts]"]
>      set gcc_output [${tool}_target_compile "" "" "none" $options]
>      if { [string match "$multi_dir\n" $gcc_output] } {
>  	pass "multilibdir $gcc_opts $multi_dir"


More information about the Gcc-patches mailing list