[binutils-gdb] x86: drop redundant calculation of EVEX broadcast size

Jan Beulich jbeulich@sourceware.org
Fri Mar 3 07:47:01 GMT 2023


https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=43da3d648ed237496f33dd620a1ee4b4f4538018

commit 43da3d648ed237496f33dd620a1ee4b4f4538018
Author: Jan Beulich <jbeulich@suse.com>
Date:   Fri Mar 3 08:46:13 2023 +0100

    x86: drop redundant calculation of EVEX broadcast size
    
    In commit a5748e0d8c50 ("x86/Intel: allow MASM representation of
    embedded broadcast") I replaced the calculation of i.broadcast.bytes in
    check_VecOperands() not paying attention to the immediately following
    call to get_broadcast_bytes() doing exactly that (again) first thing.

Diff:
---
 gas/config/tc-i386.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c
index a9314f560b7..9ceef278008 100644
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -6409,9 +6409,6 @@ check_VecOperands (const insn_template *t)
 	  return 1;
 	}
 
-      if (i.broadcast.type)
-	i.broadcast.bytes = ((1 << (t->opcode_modifier.broadcast - 1))
-			     * i.broadcast.type);
       operand_type_set (&type, 0);
       switch (get_broadcast_bytes (t, false))
 	{


More information about the Binutils-cvs mailing list