Re: Run ThreadSanitizer test only on x86/linux (issue 5437087)

[dnovillo]

On 2011/11/30 14:08:34, dvyukov wrote:

The patch is for google-main branch.
Add directives to run ThreadSanitizer tests
only on i386/x86_64-*-linux targets.

Index: gcc/ChangeLog.google-main
===================================================================
--- gcc/ChangeLog.google-main   (revision 181841)
+++ gcc/ChangeLog.google-main   (working copy)
@@ -1,3 +1,11 @@
+2011-11-30   Dmitriy Vyukov  <mailto:dvyu...@google.com>
+
+       Add directives to run ThreadSanitizer tests
+       only on i386/x86_64-*-linux targets.
+       * gcc/testsuite/gcc.dg/tsan-ignore.c: Add the directives.
+       * gcc/testsuite/gcc.dg/tsan-stack.c: Add the directives.
+       * gcc/testsuite/gcc.dg/tsan-mop.c: Add the directives.
+
  2011-11-23   Teresa Johnson  <mailto:tejohn...@google.com>

        * loop-unroll.c (loop_has_FP_comp): New function.
Index: gcc/testsuite/gcc.dg/tsan-ignore.c
===================================================================
--- gcc/testsuite/gcc.dg/tsan-ignore.c  (revision 181841)
+++ gcc/testsuite/gcc.dg/tsan-ignore.c  (working copy)
@@ -1,4 +1,4 @@
-/* { dg-do run } */
+/* { dg-do run { target i?86-*-linux* x86_64-*-linux* } } */
  /* { dg-options "-ftsan -O1 -ftsan-ignore=tsan-ignore.ignore" } */
  #include "tsan.h"
  #include "tsan-ignore.h"
Index: gcc/testsuite/gcc.dg/tsan-stack.c
===================================================================
--- gcc/testsuite/gcc.dg/tsan-stack.c   (revision 181841)
+++ gcc/testsuite/gcc.dg/tsan-stack.c   (working copy)
@@ -1,4 +1,4 @@
-/* { dg-do run } */
+/* { dg-do run { target i?86-*-linux* x86_64-*-linux* } } */
  /* { dg-options "-ftsan -O1" } */
  #include "tsan.h"

Index: gcc/testsuite/gcc.dg/tsan-mop.c
===================================================================
--- gcc/testsuite/gcc.dg/tsan-mop.c     (revision 181841)
+++ gcc/testsuite/gcc.dg/tsan-mop.c     (working copy)
@@ -1,4 +1,4 @@
-/* { dg-do run } */
+/* { dg-do run { target i?86-*-linux* x86_64-*-linux* } } */
  /* { dg-options "-ftsan -O1" } */
  #include "tsan.h"

--
This patch is available for review at

http://codereview.appspot.com/5437087

This is OK.


Diego.

http://codereview.appspot.com/5437087/