Re: [PATCH] testsuite: gluefile file need to be prefixed

[Richard Sandiford via Gcc-patches]

Torbjörn SVENSSON via Gcc-patches <gcc-patches@gcc.gnu.org> writes:
> PR/95720
> When the status wrapper is used, the gluefile need to be prefixed with
> -Wl, in order for the test cases to have the dump files with the
> expected names.
>
> gcc/testsuite/ChangeLog:
>
>         * gcc/testsuite/lib/g++.exp: Moved gluefile block to after
>           flags have been prefixed for the target_compile call.
>         * gcc/testsuite/lib/gcc.exp: Likewise.
>         * gcc/testsuite/lib/wrapper.exp: Reset adjusted state flag.

I agree this looks right, but I'm curious how you ended up with the
behaviour seen in comment 16.  I thought having:

    if { [target_info needs_status_wrapper] != "" && [info exists gluefile] } {
        lappend options "libs=${gluefile}"
        lappend options "ldflags=${wrap_flags}"
    }

before the dg-additional-files-options/gcc_adjusted_linker_flags would
lead to the gluefile missing a "-Wl," first time round.  How is it that
the gluefile still gets the "-Wl," and the -T misses it instead?

Thanks,
Richard

> Co-Authored-By: Yvan ROUX <yvan.r...@foss.st.com>
> Signed-off-by: Torbjörn SVENSSON <torbjorn.svens...@foss.st.com>
> ---
>  gcc/testsuite/lib/g++.exp     | 10 +++++-----
>  gcc/testsuite/lib/gcc.exp     | 21 +++++++++++----------
>  gcc/testsuite/lib/wrapper.exp |  7 ++++++-
>  3 files changed, 22 insertions(+), 16 deletions(-)
>
> diff --git a/gcc/testsuite/lib/g++.exp b/gcc/testsuite/lib/g++.exp
> index 24ef068b239..16e61fb4ad4 100644
> --- a/gcc/testsuite/lib/g++.exp
> +++ b/gcc/testsuite/lib/g++.exp
> @@ -303,11 +303,6 @@ proc g++_target_compile { source dest type options } {
>      global flags_to_postpone
>      global board_info
>  
> -    if { [target_info needs_status_wrapper] != "" && [info exists gluefile] 
> } {
> -     lappend options "libs=${gluefile}"
> -     lappend options "ldflags=${wrap_flags}"
> -    }
> -
>      global TEST_EXTRA_LIBS
>      if [info exists TEST_EXTRA_LIBS] {
>       lappend options "ldflags=$TEST_EXTRA_LIBS"
> @@ -333,6 +328,11 @@ proc g++_target_compile { source dest type options } {
>  
>      set options [dg-additional-files-options $options $source]
>  
> +    if { [target_info needs_status_wrapper] != "" && [info exists gluefile] 
> } {
> +     lappend options "libs=${gluefile}"
> +     lappend options "ldflags=${wrap_flags}"
> +    }
> +
>      set result [target_compile $source $dest $type $options]
>  
>      if {[board_info $tboard exists multilib_flags]} {
> diff --git a/gcc/testsuite/lib/gcc.exp b/gcc/testsuite/lib/gcc.exp
> index 1b25ebec4cf..2f145d0fdf4 100644
> --- a/gcc/testsuite/lib/gcc.exp
> +++ b/gcc/testsuite/lib/gcc.exp
> @@ -129,16 +129,6 @@ proc gcc_target_compile { source dest type options } {
>      global flags_to_postpone
>      global board_info
>  
> -    if {[target_info needs_status_wrapper] != "" && \
> -         [target_info needs_status_wrapper] != "0" && \
> -         [info exists gluefile] } {
> -     lappend options "libs=${gluefile}"
> -     lappend options "ldflags=$wrap_flags"
> -     if { $type == "executable" } {
> -         set options [concat "{additional_flags=-dumpbase \"\"}" $options]
> -     }
> -    }
> -
>      global TEST_EXTRA_LIBS
>      if [info exists TEST_EXTRA_LIBS] {
>       lappend options "ldflags=$TEST_EXTRA_LIBS"
> @@ -170,6 +160,17 @@ proc gcc_target_compile { source dest type options } {
>      lappend options "timeout=[timeout_value]"
>      lappend options "compiler=$GCC_UNDER_TEST"
>      set options [dg-additional-files-options $options $source]
> +
> +    if {[target_info needs_status_wrapper] != "" && \
> +         [target_info needs_status_wrapper] != "0" && \
> +         [info exists gluefile] } {
> +     lappend options "libs=${gluefile}"
> +     lappend options "ldflags=$wrap_flags"
> +     if { $type == "executable" } {
> +         set options [concat "{additional_flags=-dumpbase \"\"}" $options]
> +     }
> +    }
> +
>      set return_val [target_compile $source $dest $type $options]
>  
>      if {[board_info $tboard exists multilib_flags]} {
> diff --git a/gcc/testsuite/lib/wrapper.exp b/gcc/testsuite/lib/wrapper.exp
> index 5a601b269da..4a7d56941fc 100644
> --- a/gcc/testsuite/lib/wrapper.exp
> +++ b/gcc/testsuite/lib/wrapper.exp
> @@ -22,7 +22,7 @@
>  # the compiler when compiling FILENAME.
>  
>  proc ${tool}_maybe_build_wrapper { filename args } {
> -    global gluefile wrap_flags
> +    global gluefile wrap_flags gcc_adjusted_linker_flags
>  
>      if { [target_info needs_status_wrapper] != "" \
>        && [target_info needs_status_wrapper] != "0" \
> @@ -43,6 +43,11 @@ proc ${tool}_maybe_build_wrapper { filename args } {
>       if { $result != "" } {
>           set gluefile [lindex $result 0]
>           set wrap_flags [lindex $result 1]
> +
> +         # Reset the cached state of the adjusted flags
> +         if { [info exists gcc_adjusted_linker_flags] } {
> +             set gcc_adjusted_linker_flags 0
> +         }
>       }
>      }
>  }