[PATCH] SCFI: correct test names

Indu Bhagat indu.bhagat@oracle.com
Fri Feb 9 07:29:48 GMT 2024


On 2/8/24 22:58, Jan Beulich wrote:
> Having multiple tests with the same name is confusing.
> ---
> I only noticed these because I had to make further adjustments to the
> overly strict expectations, to accommodate a local change of mine
> (adding
> 
> #define EH_FRAME_ALIGNMENT 2
> 
> to tc-i386.h). I've yet to decide whether I actually want to upstream
> that change.
> 
> --- a/gas/testsuite/gas/scfi/x86_64/scfi-indirect-mov-4.d
> +++ b/gas/testsuite/gas/scfi/x86_64/scfi-indirect-mov-4.d
> @@ -1,7 +1,7 @@
>   #as: --scfi=experimental -W
>   #as:
>   #objdump: -Wf
> -#name: Synthesize CFI for indirect mem op to stack 3
> +#name: Synthesize CFI for indirect mem op to stack 4
>   #...
>   Contents of the .eh_frame section:
>   
> --- a/gas/testsuite/gas/scfi/x86_64/scfi-pushsection-2.d
> +++ b/gas/testsuite/gas/scfi/x86_64/scfi-pushsection-2.d
> @@ -1,7 +1,7 @@
>   #as: --scfi=experimental -W
>   #as:
>   #objdump: -Wf
> -#name: Synthesize CFI for demarcated code blocks 2
> +#name: Synthesize CFI with pushsection 2
>   #...
>   Contents of the .eh_frame section:
>   

Thanks for fixing these.

Indu


More information about the Binutils mailing list