public inbox for gcc-patches@gcc.gnu.org
 help / color / mirror / Atom feed
* [testsuite] ARM: require arm_thumb2_ok, drop -march
@ 2011-06-21 21:25 Janis Johnson
  2011-06-21 22:26 ` Ramana Radhakrishnan
  0 siblings, 1 reply; 2+ messages in thread
From: Janis Johnson @ 2011-06-21 21:25 UTC (permalink / raw)
  To: gcc-patches

[-- Attachment #1: Type: text/plain, Size: 180 bytes --]

Modify three tests that require thumb2 support to check for that rather
than specifying an arch value that might be overridden by multilib flags.

OK for trunk, and later for 4.6?

[-- Attachment #2: gcc-20110621-5 --]
[-- Type: text/plain, Size: 1529 bytes --]

2011-06-21  Janis Johnson  <janisjo@codesourcery.com>
	
	* gcc.target/arm/ctz.c: Don't specify -march, require thumb2.
	* gcc.target/arm/pr42879.c: Likewise.
	* gcc.target/arm/pr45701-3.c: Likewise.

Index: gcc.target/arm/ctz.c
===================================================================
--- gcc.target/arm/ctz.c	(revision 175269)
+++ gcc.target/arm/ctz.c	(working copy)
@@ -1,6 +1,6 @@
 /* { dg-do compile } */
-/* { dg-require-effective-target arm32 } */
-/* { dg-options "-O2 -march=armv6t2" } */
+/* { dg-require-effective-target arm_thumb2_ok } */
+/* { dg-options "-O2" } */
 
 unsigned int functest(unsigned int x)
 {
Index: gcc.target/arm/pr42879.c
===================================================================
--- gcc.target/arm/pr42879.c	(revision 175269)
+++ gcc.target/arm/pr42879.c	(working copy)
@@ -1,4 +1,5 @@
-/* { dg-options "-march=armv7-a -mthumb -Os" }  */
+/* { dg-require-effective-target arm_thumb2_ok } */
+/* { dg-options "-mthumb -Os" }  */
 /* { dg-final { scan-assembler "lsls" } } */
 
 struct A
Index: gcc.target/arm/pr45701-3.c
===================================================================
--- gcc.target/arm/pr45701-3.c	(revision 175269)
+++ gcc.target/arm/pr45701-3.c	(working copy)
@@ -1,5 +1,6 @@
 /* { dg-do compile } */
-/* { dg-options "-march=armv7-a -mthumb -Os" }  */
+/* { dg-require-effective-target arm_thumb2_ok } */
+/* { dg-options "-mthumb -Os" }  */
 /* { dg-final { scan-assembler "push\t.*r8" } } */
 /* { dg-final { scan-assembler-not "push\t*r3" } } */
 

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

* Re: [testsuite] ARM: require arm_thumb2_ok, drop -march
  2011-06-21 21:25 [testsuite] ARM: require arm_thumb2_ok, drop -march Janis Johnson
@ 2011-06-21 22:26 ` Ramana Radhakrishnan
  0 siblings, 0 replies; 2+ messages in thread
From: Ramana Radhakrishnan @ 2011-06-21 22:26 UTC (permalink / raw)
  To: Janis Johnson; +Cc: gcc-patches

On 21 June 2011 22:06, Janis Johnson <janisjo@codesourcery.com> wrote:
> Modify three tests that require thumb2 support to check for that rather
> than specifying an arch value that might be overridden by multilib flags.
>
> OK for trunk, and later for 4.6?

OK - this is a nice cleanup.

Ramana
>

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

end of thread, other threads:[~2011-06-21 21:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-06-21 21:25 [testsuite] ARM: require arm_thumb2_ok, drop -march Janis Johnson
2011-06-21 22:26 ` Ramana Radhakrishnan

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).