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: [graphite][patch] Add test cases for autopar's code generation part.


Hi Li,

On Sat, 2009-05-09 at 16:02 +0800, Li Feng wrote:
> This is the patch. Now the files are categorized to 4 part:
> 1) blcok-*.c with flag "-O2 -fdump-tree-graphite-all"
> 2) id-*.c with flag "-O2 -fgraphite-identity"
> 3) scop-*.c with flag "-O2 -fgraphite -fdump-tree-graphite-all"
> 4) others defaults to flag "-ansi -pedantic-errors"
> if dg-option is not set.
> 
> Now the new testsuites will missing two checkings due to
> removeing dump file checking in id-5.c and id-6.c
> gcc.dg/graphite/id-5.c scan-tree-dump-times graphite "number of SCoPs: 1" 1
> gcc.dg/graphite/id-6.c scan-tree-dump-times graphite "number of SCoPs: 1" 1

Thanks for working on this.

However I would prefer to add keep -fdump-tree-graphite-all in id*. Thus
we can add checks if we have found SCoPs. This is very necessary as
otherwise the test cases do not fail, even if they do not run through
graphite.

I just read that Sebastian asked you to do exactly the oposite. 
@Sebastian: Why do you want -fdump-tree-graphite-all to be removed?

Tobias






> 
> ChangeLog:
>     Set different default compile options by file names.
> 
>     testsuite/gcc.dg/graphite/graphite.exp: Set different default
>     compile options by file names.
>     testsuite/gcc.dg/graphite/block{0-6}.c: Remove dg-option line.
>     testsuite/gcc.dg/graphite/id-{1-9}.c: Ditto.
>     testsuite/gcc.dg/graphite/scop-{0-20}.c: Ditto.
>     testsuite/gcc.dg/graphite/scop-matmult.c: Ditto.
> 
> This is the patch:
> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
> diff --git a/gcc/testsuite/gcc.dg/graphite/graphite.exp
> b/gcc/testsuite/gcc.dg/graphite/graphite.exp
> index a125717..25af291 100644
> --- a/gcc/testsuite/gcc.dg/graphite/graphite.exp
> +++ b/gcc/testsuite/gcc.dg/graphite/graphite.exp
> @@ -23,23 +23,46 @@ if ![check_effective_target_fgraphite] {
>    return
>  }
> 
> +# Remove VALUE from LIST_VARIABLE.
> +proc lremove {list_variable value} {
> +    upvar 1 $list_variable var
> +    set idx [lsearch -exact $var $value]
> +    set var [lreplace $var $idx $idx]
> +}
> +
>  # The default action for a test is 'compile'.  Save current default.
>  global dg-do-what-default
>  set save-dg-do-what-default ${dg-do-what-default}
>  set dg-do-what-default compile
> 
> -# If a testcase doesn't have special options, use these.
> -global DEFAULT_CFLAGS
> -if ![info exists DEFAULT_CFLAGS] then {
> -    set DEFAULT_CFLAGS " -ansi -pedantic-errors"
> -}
> -
>  # Initialize `dg'.
>  dg-init
> 
>  # Main loop.
> -dg-runtest [lsort [glob -nocomplain $srcdir/$subdir/*.\[cS\]]] \
> -	"" $DEFAULT_CFLAGS
> +
> +set wait_to_run_files [lsort [glob -nocomplain $srcdir/$subdir/*.c]]
> +
> +# Flags using for block-*.c files.
> +set DEFAULT_CFLAGS_GRAPHITE_BLOCK "-O2 -fdump-tree-graphite-all"
> +set block_files [lsort [glob -nocomplain $srcdir/$subdir/block-*.c]]
> +dg-runtest $block_files "" $DEFAULT_CFLAGS_GRAPHITE_BLOCK
> +foreach block_file $block_files {lremove wait_to_run_files $block_file}
> +
> +# Flags using for id-*.c files.
> +set DEFAULT_CFLAGS_GRAPHITE_IDENTITY "-O2 -fgraphite-identity"
> +set id_files [lsort [glob -nocomplain $srcdir/$subdir/id-*.c]]
> +dg-runtest $id_files "" $DEFAULT_CFLAGS_GRAPHITE_IDENTITY
> +foreach id_file $id_files {lremove wait_to_run_files $id_file}
> +
> +# Flags using for scop-*.c files.
> +set DEFAULT_CFLAGS_GRAPHITE_SCOP "-O2 -fgraphite -fdump-tree-graphite-all"
> +set scop_files [lsort [glob -nocomplain $srcdir/$subdir/scop-*.c]]
> +dg-runtest $scop_files "" $DEFAULT_CFLAGS_GRAPHITE_SCOP
> +foreach scop_file $scop_files {lremove wait_to_run_files $scop_file}
> +
> +# Flags using for other files.
> +set DEFAULT_CFLAGS_GRAPHITE "-ansi -pedantic-errors"
> +dg-runtest $wait_to_run_files "" $DEFAULT_CFLAGS_GRAPHITE
> 
>  # Clean up.
>  set dg-do-what-default ${save-dg-do-what-default}
> diff --git a/gcc/testsuite/gcc.dg/graphite/block-0.c
> b/gcc/testsuite/gcc.dg/graphite/block-0.c
> index 31cb9f8..55b9036 100644
> --- a/gcc/testsuite/gcc.dg/graphite/block-0.c
> +++ b/gcc/testsuite/gcc.dg/graphite/block-0.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fdump-tree-graphite-all" } */
> -
>  #define N 1000
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/block-1.c
> b/gcc/testsuite/gcc.dg/graphite/block-1.c
> index e097dcf..ba772b3 100644
> --- a/gcc/testsuite/gcc.dg/graphite/block-1.c
> +++ b/gcc/testsuite/gcc.dg/graphite/block-1.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fdump-tree-graphite-all" } */
> -
>  #define MAX 8192
> 
>  void bar (void);
> diff --git a/gcc/testsuite/gcc.dg/graphite/block-2.c
> b/gcc/testsuite/gcc.dg/graphite/block-2.c
> index d5ed485..210ea34 100644
> --- a/gcc/testsuite/gcc.dg/graphite/block-2.c
> +++ b/gcc/testsuite/gcc.dg/graphite/block-2.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fdump-tree-graphite-all" } */
> -
>  typedef unsigned char   UChar;
>  typedef int             Int32;
>  typedef unsigned int    UInt32;
> diff --git a/gcc/testsuite/gcc.dg/graphite/block-3.c
> b/gcc/testsuite/gcc.dg/graphite/block-3.c
> index 188df26..6358bce 100644
> --- a/gcc/testsuite/gcc.dg/graphite/block-3.c
> +++ b/gcc/testsuite/gcc.dg/graphite/block-3.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fdump-tree-graphite-all" } */
> -
>  #define N 24
>  #define M 100
> 
> diff --git a/gcc/testsuite/gcc.dg/graphite/block-4.c
> b/gcc/testsuite/gcc.dg/graphite/block-4.c
> index 5bdd564..773dfef 100644
> --- a/gcc/testsuite/gcc.dg/graphite/block-4.c
> +++ b/gcc/testsuite/gcc.dg/graphite/block-4.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fdump-tree-graphite-all" } */
> -
>  #define N 24
>  #define M 1000
> 
> diff --git a/gcc/testsuite/gcc.dg/graphite/block-5.c
> b/gcc/testsuite/gcc.dg/graphite/block-5.c
> index c67f808..7864f32 100644
> --- a/gcc/testsuite/gcc.dg/graphite/block-5.c
> +++ b/gcc/testsuite/gcc.dg/graphite/block-5.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fdump-tree-graphite-all" } */
> -
>  #define N 10000
>  void foo (int);
>  int test ()
> diff --git a/gcc/testsuite/gcc.dg/graphite/block-6.c
> b/gcc/testsuite/gcc.dg/graphite/block-6.c
> index 613dfd1..5545ec9 100644
> --- a/gcc/testsuite/gcc.dg/graphite/block-6.c
> +++ b/gcc/testsuite/gcc.dg/graphite/block-6.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fdump-tree-graphite-all" } */
> -
>  #define N 10000
>  void foo (int);
>  int test ()
> diff --git a/gcc/testsuite/gcc.dg/graphite/id-1.c
> b/gcc/testsuite/gcc.dg/graphite/id-1.c
> index 177d2cc..70b477a 100644
> --- a/gcc/testsuite/gcc.dg/graphite/id-1.c
> +++ b/gcc/testsuite/gcc.dg/graphite/id-1.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite-identity" } */
> -
>  typedef int *lambda_vector;
>  typedef lambda_vector *lambda_matrix;
>  lambda_vector_add_mc (lambda_vector vec1, int const1,
> diff --git a/gcc/testsuite/gcc.dg/graphite/id-2.c
> b/gcc/testsuite/gcc.dg/graphite/id-2.c
> index 2a95cab..c11f6a2 100644
> --- a/gcc/testsuite/gcc.dg/graphite/id-2.c
> +++ b/gcc/testsuite/gcc.dg/graphite/id-2.c
> @@ -1,4 +1,3 @@
> -/* { dg-options "-O2" } */
>  typedef _Complex float GFC_COMPLEX_4;
>  matmul_c4 ()
>  {
> diff --git a/gcc/testsuite/gcc.dg/graphite/id-3.c
> b/gcc/testsuite/gcc.dg/graphite/id-3.c
> index 20e835d..608c1c3 100644
> --- a/gcc/testsuite/gcc.dg/graphite/id-3.c
> +++ b/gcc/testsuite/gcc.dg/graphite/id-3.c
> @@ -1,4 +1,3 @@
> -/* { dg-options "-O2" } */
>  struct {
>  }
>  mmaxloc0_4_i1 ()
> diff --git a/gcc/testsuite/gcc.dg/graphite/id-4.c
> b/gcc/testsuite/gcc.dg/graphite/id-4.c
> index 4aa72a3..38f6738 100644
> --- a/gcc/testsuite/gcc.dg/graphite/id-4.c
> +++ b/gcc/testsuite/gcc.dg/graphite/id-4.c
> @@ -1,4 +1,3 @@
> -/* { dg-options "-O2" } */
>  extern a[];
>  g ()
>  {
> diff --git a/gcc/testsuite/gcc.dg/graphite/id-5.c
> b/gcc/testsuite/gcc.dg/graphite/id-5.c
> index d948ff3..8a82ebe 100644
> --- a/gcc/testsuite/gcc.dg/graphite/id-5.c
> +++ b/gcc/testsuite/gcc.dg/graphite/id-5.c
> @@ -1,4 +1,3 @@
> -/* { dg-options "-O2 -fdump-tree-graphite-all" } */
>  void matmul_i1 ()
>  {
>    int *abase;
> @@ -13,5 +12,3 @@ void matmul_i1 ()
>  	dest_y[x] += abase_n[x];
>      }
>  }
> -/* { dg-final { scan-tree-dump-times "number of SCoPs: 1" 1 "graphite"} } */
> -/* { dg-final { cleanup-tree-dump "graphite" } } */
> diff --git a/gcc/testsuite/gcc.dg/graphite/id-6.c
> b/gcc/testsuite/gcc.dg/graphite/id-6.c
> index 4101ea0..6ad8b6e 100644
> --- a/gcc/testsuite/gcc.dg/graphite/id-6.c
> +++ b/gcc/testsuite/gcc.dg/graphite/id-6.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fdump-tree-graphite-all" } */
> -
>  #define N 10000
>  void foo (int);
>  int test ()
> @@ -29,6 +27,3 @@ int test ()
> 
>    return a[20];
>  }
> -
> -/* { dg-final { scan-tree-dump-times "number of SCoPs: 1" 1 "graphite"} } */
> -/* { dg-final { cleanup-tree-dump "graphite" } } */
> diff --git a/gcc/testsuite/gcc.dg/graphite/id-7.c
> b/gcc/testsuite/gcc.dg/graphite/id-7.c
> index fdca535..9fa811d 100644
> --- a/gcc/testsuite/gcc.dg/graphite/id-7.c
> +++ b/gcc/testsuite/gcc.dg/graphite/id-7.c
> @@ -1,4 +1,3 @@
> -/* { dg-options "-Os -fdump-tree-graphite-all" } */
>  void foo (int *BM_tab)
>  {
>    int *BM_tab_base;
> @@ -15,4 +14,3 @@ int main ()
>    foo (BM_tab);
>    return 0;
>  }
> -/* { dg-final { cleanup-tree-dump "graphite" } } */
> diff --git a/gcc/testsuite/gcc.dg/graphite/id-8.c
> b/gcc/testsuite/gcc.dg/graphite/id-8.c
> index 37384a8..1a278c1 100644
> --- a/gcc/testsuite/gcc.dg/graphite/id-8.c
> +++ b/gcc/testsuite/gcc.dg/graphite/id-8.c
> @@ -1,4 +1,3 @@
> -/* { dg-options "-Os -fdump-tree-graphite-all" } */
>  int blah;
>  foo()
>  {
> @@ -13,4 +12,3 @@ foo()
>      }
>    return blah;
>  }
> -/* { dg-final { cleanup-tree-dump "graphite" } } */
> diff --git a/gcc/testsuite/gcc.dg/graphite/id-9.c
> b/gcc/testsuite/gcc.dg/graphite/id-9.c
> index af72c4d..2199538 100644
> --- a/gcc/testsuite/gcc.dg/graphite/id-9.c
> +++ b/gcc/testsuite/gcc.dg/graphite/id-9.c
> @@ -1,4 +1,3 @@
> -/* { dg-options "-O2" } */
>  typedef enum
>  {
>    no_op,
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-0.c
> b/gcc/testsuite/gcc.dg/graphite/scop-0.c
> index ea3ae06..067e7bc 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-0.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-0.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  int foo (void);
>  void bar (void);
> 
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-1.c
> b/gcc/testsuite/gcc.dg/graphite/scop-1.c
> index ed6159f..ba2590c 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-1.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-1.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  void bar (void);
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-10.c
> b/gcc/testsuite/gcc.dg/graphite/scop-10.c
> index 8aff2c7..139f4c1 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-10.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-10.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  void bar (void);
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-11.c
> b/gcc/testsuite/gcc.dg/graphite/scop-11.c
> index e5a0fdb..544b36b 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-11.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-11.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  void bar ();
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-12.c
> b/gcc/testsuite/gcc.dg/graphite/scop-12.c
> index 0c13033..71d34b1 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-12.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-12.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  void bar ();
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-13.c
> b/gcc/testsuite/gcc.dg/graphite/scop-13.c
> index aa55e10..b2ca5b4 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-13.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-13.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  void bar ();
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-14.c
> b/gcc/testsuite/gcc.dg/graphite/scop-14.c
> index a707b01..867e293 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-14.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-14.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  void bar ();
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-15.c
> b/gcc/testsuite/gcc.dg/graphite/scop-15.c
> index 9324631..6b2c2bd 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-15.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-15.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  #  define EXTERN(type, array)  extern type array[]
>  typedef unsigned char  uch;
>  typedef unsigned short ush;
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-16.c
> b/gcc/testsuite/gcc.dg/graphite/scop-16.c
> index d0b32f6..d465d95 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-16.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-16.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  #define N 10000
>  void foo (int);
>  int test ()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-17.c
> b/gcc/testsuite/gcc.dg/graphite/scop-17.c
> index c2fec3f..dd7bdad 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-17.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-17.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  #define N 10000
>  void foo (int);
>  int test ()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-18.c
> b/gcc/testsuite/gcc.dg/graphite/scop-18.c
> index 6264116..7d56cb7 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-18.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-18.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  #define N 24
>  #define M 1000
> 
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-19.c
> b/gcc/testsuite/gcc.dg/graphite/scop-19.c
> index 3ad4997..cfbf401 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-19.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-19.c
> @@ -1,4 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
>  typedef unsigned int __uint32_t;
>  typedef __uint32_t __size_t;
>  typedef __size_t size_t;
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-2.c
> b/gcc/testsuite/gcc.dg/graphite/scop-2.c
> index cf25dcd..9e494f4 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-2.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-2.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  void bar (void);
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-20.c
> b/gcc/testsuite/gcc.dg/graphite/scop-20.c
> index 470812f..6e71df3 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-20.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-20.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  void bar (void);
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-3.c
> b/gcc/testsuite/gcc.dg/graphite/scop-3.c
> index efcaa6c..e20bfdc 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-3.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-3.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  int toto()
>  {
>    int i, j, k;
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-4.c
> b/gcc/testsuite/gcc.dg/graphite/scop-4.c
> index 515c53a..4ab2d07 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-4.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-4.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  void bar ();
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-5.c
> b/gcc/testsuite/gcc.dg/graphite/scop-5.c
> index 697a28e..4f4b45b 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-5.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-5.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  void bar ();
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-6.c
> b/gcc/testsuite/gcc.dg/graphite/scop-6.c
> index d262320..df208ac 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-6.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-6.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  void bar (void);
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-7.c
> b/gcc/testsuite/gcc.dg/graphite/scop-7.c
> index 1187ce1..c02748f 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-7.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-7.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  void bar (void);
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-8.c
> b/gcc/testsuite/gcc.dg/graphite/scop-8.c
> index 491ad37..3f14e34 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-8.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-8.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  int bar (void);
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-9.c
> b/gcc/testsuite/gcc.dg/graphite/scop-9.c
> index 871b86b..a803d92 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-9.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-9.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  void bar (void);
> 
>  int toto()
> diff --git a/gcc/testsuite/gcc.dg/graphite/scop-matmult.c
> b/gcc/testsuite/gcc.dg/graphite/scop-matmult.c
> index 61a5be1..259c86c 100644
> --- a/gcc/testsuite/gcc.dg/graphite/scop-matmult.c
> +++ b/gcc/testsuite/gcc.dg/graphite/scop-matmult.c
> @@ -1,5 +1,3 @@
> -/* { dg-options "-O2 -fgraphite -fdump-tree-graphite-all" } */
> -
>  float A[1000][1000], B[1000][1000], C[1000][1000];
> 
>  /* Multiply two n x n matrices A and B and store the result in C.  */
> 
> 
> On Thu, May 7, 2009 at 6:42 PM, Sebastian Pop <sebpop@gmail.com> wrote:
> > Hi Li,
> >
> > Your new patch looks good.  Please apply to the graphite branch.
> >
> > On Wed, May 6, 2009 at 21:45, Li Feng <nemokingdom@gmail.com> wrote:
> >> 1. We can filter the testcases in graphite/block-*.c
> >> 2. We can't filter the testcases in graphite/id-*.c, because their
> >>    option differs (about 3 different option or more).
> >> So we just filter the block-*.c?
> >
> > No, we should change the options in the id-*.c files.
> >
> > In particular for id-{6,7,8}.c please remove scan-tree-dump-times and
> > the -fdump-tree-* flags.
> >
> > All these files should be flagged with "-O2 -fgraphite-identity".
> > Right now these flags are equivalent to "-O2", as the graphite
> > identity transform is enabled at -O2 and higher in the branch.
> >
> > For the scop-*.c files you may also remove the dg-options line
> > and use these options "-O2 -fgraphite -fdump-tree-graphite-all"
> > in graphite.exp.
> >
> > Thanks for working on this,
> > Sebastian
> >


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