This is the mail archive of the binutils@sourceware.org mailing list for the binutils project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 1/2] x86: drop Vec_Disp8

On Wed, Nov 29, 2017 at 6:20 AM, Jan Beulich <JBeulich@suse.com> wrote:
> x86: drop Vec_Disp8
>
> This is fully redundant with Disp8MemShift being non-zero, and hence can
> be folded with normal Disp8 handling.
>
> gas/
> 2017-11-29  Jan Beulich  <jbeulich@suse.com>
>
>         * config/tc-i386.c (enum i386_error): Remove try_vector_disp8.
>         (mode_from_disp_size, build_modrm_byte, build_modrm_byte,
>         disp_size): Remove reference to .vec_disp8.
>         (output_disp): Likewise. Unconditionally use i.memshift.
>         (fits_in_vec_disp8): Rename to fits_in_disp8.
>         (type_names): Remove OPERAND_TYPE_VEC_DISP8 entry.
>         (optimize_disp): Use fits_in_disp8.
>         (check_VecOperands): Re-work (simplify) .disp8memshift
>         conditional handling.
>
> opcodes/
> 2017-11-29  Jan Beulich  <jbeulich@suse.com>
>
>         * i386-gen.c (operand_type_init): Remove OPERAND_TYPE_VEC_DISP8
>         entry.
>         (operand_types): Remove Vec_Disp8 entry.
>         * i386-opc.h (Vec_Disp8): Delete.
>         (union i386_operand_type): Remove vec_disp8.
>         (i386-opc.tbl): Remove Vec_Disp8.
>         * i386-init.h, i386-tbl.h: Re-generate.

OK if there are no regressions on i686 and x86-64.

Thanks.

-- 
H.J.
Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]