public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* PATCH: Check ia32 instead of ilp32
@ 2011-09-10 20:55 H.J. Lu
  2011-09-10 21:09 ` Uros Bizjak
  0 siblings, 1 reply; 3+ messages in thread
From: H.J. Lu @ 2011-09-10 20:55 UTC (permalink / raw)
  To: gcc-patches; +Cc: Uros Bizjak

Hi,

On x86, x32 PIC is the same as 64bit PIC.  This patch checks ia32 instead
of ilp32.  Tested on Linux/x86-64 with -m32, -m64 and -mx32.  OK for
trunk?

Thanks.


H.J.
---
2011-09-10  H.J. Lu  <hongjiu.lu@intel.com>

	* gcc.dg/sibcall-6.c: Check ia32 instead of ilp32.

diff --git a/gcc/testsuite/gcc.dg/sibcall-6.c b/gcc/testsuite/gcc.dg/sibcall-6.c
index 3dba764..b62a8ee 100644
--- a/gcc/testsuite/gcc.dg/sibcall-6.c
+++ b/gcc/testsuite/gcc.dg/sibcall-6.c
@@ -7,7 +7,7 @@
    Contributed by Andreas Bauer <baueran@in.tum.de>  */
 
 /* { dg-do run { target i?86-*-* x86_64-*-* s390*-*-* } } */
-/* { dg-skip-if "" { { i?86-*-* x86_64-*-* } && { ilp32 && { ! nonpic } } } { "*" } { "" } } */
+/* { dg-skip-if "" { { i?86-*-* x86_64-*-* } && { ia32 && { ! nonpic } } } { "*" } { "" } } */
 /* { dg-options "-O2 -foptimize-sibling-calls -fno-ipa-cp" } */
 
 extern void abort (void);

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: PATCH: Check ia32 instead of ilp32
  2011-09-10 20:55 PATCH: Check ia32 instead of ilp32 H.J. Lu
@ 2011-09-10 21:09 ` Uros Bizjak
  0 siblings, 0 replies; 3+ messages in thread
From: Uros Bizjak @ 2011-09-10 21:09 UTC (permalink / raw)
  To: H.J. Lu; +Cc: gcc-patches

On Sat, Sep 10, 2011 at 9:02 PM, H.J. Lu <hongjiu.lu@intel.com> wrote:

> On x86, x32 PIC is the same as 64bit PIC.  This patch checks ia32 instead
> of ilp32.  Tested on Linux/x86-64 with -m32, -m64 and -mx32.  OK for
> trunk?
>
> Thanks.
>
>
> H.J.
> ---
> 2011-09-10  H.J. Lu  <hongjiu.lu@intel.com>
>
>        * gcc.dg/sibcall-6.c: Check ia32 instead of ilp32.

OK.

Thanks,
Uros.

^ permalink raw reply	[flat|nested] 3+ messages in thread

* PATCH: Check ia32 instead of ilp32
@ 2011-11-14 19:18 H.J. Lu
  0 siblings, 0 replies; 3+ messages in thread
From: H.J. Lu @ 2011-11-14 19:18 UTC (permalink / raw)
  To: gcc-patches

Hi,

ilp32 includes both x32 and ia32.  -march=i486 only applies to ia32.
I checked in this patch to check ia32 instead of ilp32.


H.J.
---
Index: ChangeLog
===================================================================
--- ChangeLog	(revision 181359)
+++ ChangeLog	(working copy)
@@ -1,3 +1,7 @@
+2011-11-14  H.J. Lu  <hongjiu.lu@intel.com>
+
+	* testsuite/lib/libitm.exp: Check ia32 instead of ilp32.
+
 2011-11-10  Rainer Orth  <ro@CeBiTec.Uni-Bielefeld.DE>
 
 	* config/alpha/sjlj.S (_ITM_beginTransaction) [!__ELF__]: Don't use
Index: testsuite/lib/libitm.exp
===================================================================
--- testsuite/lib/libitm.exp	(revision 181359)
+++ testsuite/lib/libitm.exp	(working copy)
@@ -144,7 +144,7 @@ proc libitm_init { args } {
 
     # We use atomic operations in the testcases to validate results.
     if { ([istarget i?86-*-*] || [istarget x86_64-*-*])
-	 && [check_effective_target_ilp32] } {
+	 && [check_effective_target_ia32] } {
 	lappend ALWAYS_CFLAGS "additional_flags=-march=i486"
     }
 

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-11-14 18:14 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-09-10 20:55 PATCH: Check ia32 instead of ilp32 H.J. Lu
2011-09-10 21:09 ` Uros Bizjak
2011-11-14 19:18 H.J. Lu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).