[PATCH v2] gas: Move gas_sframe_check to binutils-common.exp

[H.J. Lu]

On Mon, Jul 14, 2025 at 6:01 AM Alan Modra <amodra@gmail.com> wrote:
>
> On Mon, Jul 14, 2025 at 05:17:45AM +0800, H.J. Lu wrote:
> > On Mon, Jul 14, 2025 at 5:15 AM H.J. Lu <hjl.tools@gmail.com> wrote:
> > > Here is the v2 patch to replace gas_host_run with remote_exec so
> > > that it can be used in ld tests.
> > >
> >
> > Oops.  Wrong patch.  Here is the right one.
>
> OK.
>
> > +    if { ![string match "" $comp_output]
> > +      || [string match "*sframe not supported for target*" $comp_output] } then {
>
> May as well fix this redundant check while you're at it.

Sure.  I will check the patch with

    set output_file "tmpdir/check_as_sframe.out"
    set status [remote_exec host "$AS $ASFLAGS $as_file" "2> $output_file"]
    set comp_output [file_contents "$output_file"]
    set ASFLAGS "$old_ASFLAGS"

    if { ![string match "" $comp_output] } then {
        verbose -log "SFrame not supported in AS"
        return 0
    } else {
        verbose -log "SFrame supported in AS"
        return 1
    }

Thanks.

>
> --
> Alan Modra



-- 
H.J.