Re: PATCH: Pass and return pointers in word_mode

2012-03-06 Thread Uros Bizjak
On Tue, Mar 6, 2012 at 11:45 PM, H.J. Lu  wrote:
> Pmode may not be word_mode for TARGET_64BIT.  This patch changes to
> word_mode instead of Pmode.  OK for trunk?

> 2012-03-02  H.J. Lu  
>
>        * config/i386/i386.c (function_value_64): Return pointers in
>        word_mode instead of Pmode.
>        (ix86_promote_function_mode): Likewise.

OK.

Thanks,
Uros.


PATCH: Pass and return pointers in word_mode

2012-03-06 Thread H.J. Lu
Pmode may not be word_mode for TARGET_64BIT.  This patch changes to
word_mode instead of Pmode.  OK for trunk?

Thanks.


H.J.
2012-03-02  H.J. Lu  

* config/i386/i386.c (function_value_64): Return pointers in
word_mode instead of Pmode.
(ix86_promote_function_mode): Likewise.

diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c
index 65c4c42..ac9c714 100644
--- a/gcc/config/i386/i386.c
+++ b/gcc/config/i386/i386.c
@@ -7240,8 +7240,8 @@ function_value_64 (enum machine_mode orig_mode, enum 
machine_mode mode,
 }
   else if (POINTER_TYPE_P (valtype))
 {
-  /* Pointers are always returned in Pmode. */
-  mode = Pmode;
+  /* Pointers are always returned in word_mode.  */
+  mode = word_mode;
 }
 
   ret = construct_container (mode, orig_mode, valtype, 1,
@@ -7312,7 +7312,8 @@ ix86_function_value (const_tree valtype, const_tree 
fntype_or_decl,
   return ix86_function_value_1 (valtype, fntype_or_decl, orig_mode, mode);
 }
 
-/* Pointer function arguments and return values are promoted to Pmode.  */
+/* Pointer function arguments and return values are promoted to
+   word_mode.  */
 
 static enum machine_mode
 ix86_promote_function_mode (const_tree type, enum machine_mode mode,
@@ -7322,7 +7323,7 @@ ix86_promote_function_mode (const_tree type, enum 
machine_mode mode,
   if (type != NULL_TREE && POINTER_TYPE_P (type))
 {
   *punsignedp = POINTERS_EXTEND_UNSIGNED;
-  return Pmode;
+  return word_mode;
 }
   return default_promote_function_mode (type, mode, punsignedp, fntype,
for_return);