[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