diff mbox

PATCH: PR target/52352: [x32] - Wrong code to access addresses 0x80000000 to 0xFFFFFFFF using registers

Message ID 20120224201930.GA6591@intel.com
State New
Headers show

Commit Message

H.J. Lu Feb. 24, 2012, 8:19 p.m. UTC 
Hi,

This patches enables *movabs<mode>_1 and *movabs<mode>_2 only for
TARGET_LP64 since x32 doesn't need 64bit address.  OK for trunk?

Thanks.

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

	PR target/52352
	* config/i386/i386.md (*movabs<mode>_1): Enable only for
	TARGET_LP64.
	(*movabs<mode>_2): Likewise.

Comments

Uros Bizjak Feb. 25, 2012, 8:40 a.m. UTC | #1 
On Fri, Feb 24, 2012 at 9:19 PM, H.J. Lu <hongjiu.lu@intel.com> wrote:

> This patches enables *movabs<mode>_1 and *movabs<mode>_2 only for
> TARGET_LP64 since x32 doesn't need 64bit address.  OK for trunk?
>
> Thanks.
>
> H.J.
> ---
> 2012-02-24  H.J. Lu  <hongjiu.lu@intel.com>
>
>        PR target/52352
>        * config/i386/i386.md (*movabs<mode>_1): Enable only for
>        TARGET_LP64.
>        (*movabs<mode>_2): Likewise.

OK.

Thanks,
Uros.
diff mbox

Patch

diff --git a/gcc/config/i386/i386.md b/gcc/config/i386/i386.md
index 6e2c123..0291e60 100644
--- a/gcc/config/i386/i386.md
+++ b/gcc/config/i386/i386.md
@@ -2362,7 +2362,7 @@ 
 (define_insn "*movabs<mode>_1"
   [(set (mem:SWI1248x (match_operand:DI 0 "x86_64_movabs_operand" "i,r"))
 	(match_operand:SWI1248x 1 "nonmemory_operand" "a,er"))]
-  "TARGET_64BIT && ix86_check_movabs (insn, 0)"
+  "TARGET_LP64 && ix86_check_movabs (insn, 0)"
   "@
    movabs{<imodesuffix>}\t{%1, %P0|%P0, %1}
    mov{<imodesuffix>}\t{%1, %a0|%a0, %1}"
@@ -2376,7 +2376,7 @@ 
 (define_insn "*movabs<mode>_2"
   [(set (match_operand:SWI1248x 0 "register_operand" "=a,r")
         (mem:SWI1248x (match_operand:DI 1 "x86_64_movabs_operand" "i,r")))]
-  "TARGET_64BIT && ix86_check_movabs (insn, 1)"
+  "TARGET_LP64 && ix86_check_movabs (insn, 1)"
   "@
    movabs{<imodesuffix>}\t{%P1, %0|%0, %P1}
    mov{<imodesuffix>}\t{%a1, %0|%0, %a1}"