From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 13408 invoked by alias); 9 Nov 2007 13:52:49 -0000 Received: (qmail 13397 invoked by uid 22791); 9 Nov 2007 13:52:49 -0000 X-Spam-Check-By: sourceware.org Received: from qmta05.westchester.pa.mail.comcast.net (HELO QMTA05.westchester.pa.mail.comcast.net) (76.96.62.48) by sourceware.org (qpsmtpd/0.31) with ESMTP; Fri, 09 Nov 2007 13:52:41 +0000 Received: from OMTA11.westchester.pa.mail.comcast.net ([76.96.62.36]) by QMTA05.westchester.pa.mail.comcast.net with smtp id AbNZ1Y0040mv7h00009U00; Fri, 09 Nov 2007 13:52:37 +0000 Received: from lucon.org ([24.6.230.138]) by OMTA11.westchester.pa.mail.comcast.net with comcast id Adrd1Y00P2zoa480000000; Fri, 09 Nov 2007 13:51:38 +0000 X-Authority-Analysis: v=1.0 c=1 a=K4C-M8qRYfUNH0pmmvwA:9 a=E_41fcO_6ZYwZs9OcJivVjsR2koA:4 a=dGJ0OcVc7YAA:10 a=gi0PWCVxevcA:10 Received: by lucon.org (Postfix, from userid 500) id 7EBFFF8275; Fri, 9 Nov 2007 05:51:37 -0800 (PST) Date: Fri, 09 Nov 2007 13:52:00 -0000 From: "H.J. Lu" To: binutils@sources.redhat.com Subject: PATCH: Re-order suffix check in x86 assembler Message-ID: <20071109135137.GA6112@lucon.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline User-Agent: Mutt/1.5.14 (2007-02-12) Mailing-List: contact binutils-help@sourceware.org; run by ezmlm Precedence: bulk List-Id: List-Subscribe: List-Archive: List-Post: List-Help: , Sender: binutils-owner@sourceware.org X-SW-Source: 2007-11/txt/msg00091.txt.bz2 I am checking this patch to re-order suffix check in x86 assembler. H.J. --- 2007-11-09 H.J. Lu * config/tc-i386.c (match_template): Re-order suffix check. --- gas/config/tc-i386.c.short 2007-11-01 17:30:30.000000000 -0700 +++ gas/config/tc-i386.c 2007-11-09 05:49:22.000000000 -0800 @@ -3011,13 +3011,13 @@ match_template (void) continue; /* Check the suffix, except for some instructions in intel mode. */ - if (((t->opcode_modifier.no_bsuf & suffix_check.no_bsuf) - || (t->opcode_modifier.no_wsuf & suffix_check.no_wsuf) - || (t->opcode_modifier.no_lsuf & suffix_check.no_lsuf) - || (t->opcode_modifier.no_ssuf & suffix_check.no_ssuf) - || (t->opcode_modifier.no_qsuf & suffix_check.no_qsuf) - || (t->opcode_modifier.no_ldsuf & suffix_check.no_ldsuf)) - && !(intel_syntax && t->opcode_modifier.ignoresize)) + if ((!intel_syntax || !t->opcode_modifier.ignoresize) + && ((t->opcode_modifier.no_bsuf && suffix_check.no_bsuf) + || (t->opcode_modifier.no_wsuf && suffix_check.no_wsuf) + || (t->opcode_modifier.no_lsuf && suffix_check.no_lsuf) + || (t->opcode_modifier.no_ssuf && suffix_check.no_ssuf) + || (t->opcode_modifier.no_qsuf && suffix_check.no_qsuf) + || (t->opcode_modifier.no_ldsuf && suffix_check.no_ldsuf))) continue; for (j = 0; j < MAX_OPERANDS; j++)