[PATCH] x86/Intel: use proper manifest constant in favor of literal number

Jan Beulich JBeulich@suse.com
Tue Jul 24 14:50:00 GMT 2012


Using the dedicated manifest constant is more descriptive.

2012-07-24  Jan Beulich <jbeulich@suse.com>

	* config/tc-i386-intel.c (i386_intel_simplify_register):
	Replace literal 4 by corresponding ESP_REG_NUM.

--- a/gas/config/tc-i386-intel.c
+++ b/gas/config/tc-i386-intel.c
@@ -285,7 +285,7 @@ i386_intel_simplify_register (expression
   else
     {
       /* esp is invalid as index */
-      intel_state.index = i386_regtab + REGNAM_EAX + 4;
+      intel_state.index = i386_regtab + REGNAM_EAX + ESP_REG_NUM;
     }
   return 2;
 }
@@ -440,7 +440,7 @@ static int i386_intel_simplify (expressi
 		break;
 	      default:
 		/* esp is invalid as index */
-		intel_state.index = i386_regtab + REGNAM_EAX + 4;
+		intel_state.index = i386_regtab + REGNAM_EAX + ESP_REG_NUM;
 		break;
 	      }
 



-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: binutils-mainline-x86-intel-esp-name.patch
URL: <https://sourceware.org/pipermail/binutils/attachments/20120724/daae3dec/attachment.ksh>


More information about the Binutils mailing list