This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] Add additional __builtin_unreachable() test.

From: Paolo Bonzini <paolo dot bonzini at gmail dot com>
To: David Daney <ddaney at caviumnetworks dot com>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Wed, 17 Jun 2009 18:04:05 +0200
Subject: Re: [PATCH] Add additional __builtin_unreachable() test.
References: <4A37D985.50800@caviumnetworks.com> <4A389CCC.7010401@gmail.com> <4A391337.4080002@caviumnetworks.com>

Go ahead after changing this:

/* { dg-options "-O2 -m32 -fomit-frame-pointer" } */

to:
/* { dg-require-effective-target ilp32 } */
/* { dg-options "-O2 -fomit-frame-pointer" } */
Why? Making that change results in the test being UNSUPPORTED on x86_64.

But not all gcc builds on x86_64 are biarch 32-bit/64-bit (though it's the default). Usually what you do is test both -m32 and -m64, and that change will make the test UNSUPPORTED in the -m64 run, and PASS :-) in the -m32 run.

Paolo

References:
- [PATCH] Add additional __builtin_unreachable() test.
  - From: David Daney
- Re: [PATCH] Add additional __builtin_unreachable() test.
  - From: Paolo Bonzini
- Re: [PATCH] Add additional __builtin_unreachable() test.
  - From: David Daney

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]