public inbox for binutils-cvs@sourceware.org
 help / color / mirror / Atom feed
* [binutils-gdb] x86: drop redundant calculation of EVEX broadcast size
@ 2023-03-03  7:47 Jan Beulich
  0 siblings, 0 replies; only message in thread
From: Jan Beulich @ 2023-03-03  7:47 UTC (permalink / raw)
  To: bfd-cvs

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))
 	{

^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2023-03-03  7:47 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-03-03  7:47 [binutils-gdb] x86: drop redundant calculation of EVEX broadcast size Jan Beulich

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).