[PATCH v2] Revert "x86: Replace evex-no-scale.s with evex-no-scale-[32|64].s"
H.J. Lu
hjl.tools@gmail.com
Mon Jul 20 11:47:58 GMT 2020
On Mon, Jul 20, 2020 at 12:01 AM Jan Beulich <jbeulich@suse.com> wrote:
>
> This reverts commit 19449d7c67690c641b1ec9c13ff3531677a5afcc, addressing
> the issue that was run into back then: There was no relationship to i686-*
> and/or cross builds on 64-bit hosts. The sole problem was the use of / as
> as comment character in certain ELF targets. Instead of division, use a
> comparison operation.
>
> At the same time also revert the ELF related part of 99c2d522f7a7 ("x86:
> Update assembler tests for non-ELF targets") by replacing the construct
> that's problematic for non-ELF, and by adding the "#pass" patterns to
> the expected output files to cover for the tail padding generated into
> COFF output.
>
> gas/
> 2020-07-XX Jan Beulich <jbeulich@suse.com>
>
> * testsuite/gas/i386/evex-no-scale-32.d,
> testsuite/gas/i386/evex-no-scale-64.d: Add #source and #pass.
> * testsuite/gas/i386/evex-no-scale-32.s,
> testsuite/gas/i386/evex-no-scale-64.s: Rename / fold into ...
> * testsuite/gas/i386/evex-no-scale.s: ... this. Use .struct
> instead of .section.
> * testsuite/gas/i386/i386.exp: Move above tests out of ELF-
> specific section.
>
OK.
Thanks.
--
H.J.
More information about the Binutils
mailing list