Patchwork [committed] PATCH: Fix a comment typo in ix86_split_lea_for_addr

login
register
mail settings
Submitter H.J. Lu
Date Jan. 18, 2014, 2:53 a.m.
Message ID <20140118025334.GA680@lucon.org>
Download mbox | patch
Permalink /patch/312259/
State New
Headers show

Comments

H.J. Lu - Jan. 18, 2014, 2:53 a.m.
Hi,

I checked in this patch as an obvious fix to correct a comment typo
in ix86_split_lea_for_addr.  The line below the comments is

gcc_assert (regno2 != regno0);

There is no way for r1 = r1 + C * r1.  It should be r1 = r1 + C * r2.

H.J.
---

Patch

Index: ChangeLog
===================================================================
--- ChangeLog	(revision 206744)
+++ ChangeLog	(working copy)
@@ -1,3 +1,8 @@ 
+2014-01-17  H.J. Lu  <hongjiu.lu@intel.com>
+
+	* config/i386/i386.c (ix86_split_lea_for_addr): Fix a comment
+	typo.
+
 2014-01-17  John David Anglin  <danglin@gcc.gnu.org>
 
 	* config/pa/pa.c (pa_attr_length_indirect_call): Don't output a short
Index: config/i386/i386.c
===================================================================
--- config/i386/i386.c	(revision 206744)
+++ config/i386/i386.c	(working copy)
@@ -18309,7 +18309,7 @@  ix86_split_lea_for_addr (rtx insn, rtx o
       /* Case r1 = r1 + ...  */
       if (regno1 == regno0)
 	{
-	  /* If we have a case r1 = r1 + C * r1 then we
+	  /* If we have a case r1 = r1 + C * r2 then we
 	     should use multiplication which is very
 	     expensive.  Assume cost model is wrong if we
 	     have such case here.  */