This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
PATCH: Move the first i.error out of the loop
- From: "H.J. Lu" <hongjiu dot lu at intel dot com>
- To: binutils at sourceware dot org
- Date: Wed, 4 Aug 2010 13:53:26 -0700
- Subject: PATCH: Move the first i.error out of the loop
- Reply-to: "H.J. Lu" <hjl dot tools at gmail dot com>
Hi,
I checked in this small optimization.
H.J.
---
diff --git a/gas/ChangeLog b/gas/ChangeLog
index 648e61b..54a88f6 100644
--- a/gas/ChangeLog
+++ b/gas/ChangeLog
@@ -1,3 +1,8 @@
+2010-08-04 H.J. Lu <hongjiu.lu@intel.com>
+
+ * config/tc-i386.c (match_template): Move the first i.error
+ out of the loop.
+
2010-08-04 Alan Modra <amodra@gmail.com>
* configure.tgt (m32c): Set endian=little.
diff --git a/gas/config/tc-i386.c b/gas/config/tc-i386.c
index 281faaa..03ea562 100644
--- a/gas/config/tc-i386.c
+++ b/gas/config/tc-i386.c
@@ -3848,12 +3848,13 @@ match_template (void)
else if (i.suffix == LONG_DOUBLE_MNEM_SUFFIX)
suffix_check.no_ldsuf = 1;
+ /* Must have right number of operands. */
+ i.error = number_of_operands_mismatch;
+
for (t = current_templates->start; t < current_templates->end; t++)
{
addr_prefix_disp = -1;
- /* Must have right number of operands. */
- i.error = number_of_operands_mismatch;
if (i.operands != t->operands)
continue;