diff mbox

[02/10] addr32: Only handle zero-extended DImode addresses

Message ID 20120302203833.GB2179@intel.com
State New
Headers show

Commit Message

H.J. Lu March 2, 2012, 8:38 p.m. UTC 
Hi,

We only need to handle zero-extended addresses in DImode.
OK for trunk?

BTW, all my 10 addr32 patches are tested on Linux/x86-64.

Thanks.


H.J.
---
2012-03-02  H.J. Lu  <hongjiu.lu@intel.com>

	* config/i386/i386.c (ix86_print_operand_address): Only handle
	zero-extended DImode addresses.

Comments

Uros Bizjak March 4, 2012, 8:13 p.m. UTC | #1 
On Fri, Mar 2, 2012 at 9:38 PM, H.J. Lu <hongjiu.lu@intel.com> wrote:

> We only need to handle zero-extended addresses in DImode.
> OK for trunk?

> 2012-03-02  H.J. Lu  <hongjiu.lu@intel.com>
>
>        * config/i386/i386.c (ix86_print_operand_address): Only handle
>        zero-extended DImode addresses.

OK.

Thanks,
Uros.
Uros Bizjak March 8, 2012, 3:20 p.m. UTC | #2 
On Sun, Mar 4, 2012 at 9:13 PM, Uros Bizjak <ubizjak@gmail.com> wrote:

>> We only need to handle zero-extended addresses in DImode.
>> OK for trunk?
>
>> 2012-03-02  H.J. Lu  <hongjiu.lu@intel.com>
>>
>>        * config/i386/i386.c (ix86_print_operand_address): Only handle
>>        zero-extended DImode addresses.
>
> OK.

The patch was reverted due to PR target/52530.

Uros.
diff mbox

Patch

diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index 2cbfb64..7cb8fda 100644
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -14511,7 +14511,7 @@  ix86_print_operand_address (FILE *file, rtx addr)
 
       /* Print SImode registers for zero-extended addresses to force
 	 addr32 prefix.  Otherwise print DImode registers to avoid it.  */
-      if (TARGET_64BIT)
+      if (TARGET_64BIT && GET_MODE (addr) == DImode)
 	code = ((GET_CODE (addr) == ZERO_EXTEND
 		 || GET_CODE (addr) == AND)
 		? 'l'