mirror of
git://gcc.gnu.org/git/gcc.git
synced 2025-02-10 10:19:51 +08:00
re PR tree-optimization/36009 (PRE causes missed loop store motion, store sinking doesn't work)
2008-05-15 Richard Guenther <rguenther@suse.de> PR tree-optimization/36009 PR tree-optimization/36204 * tree-ssa-loop-im.c (tree-ssa-propagate.h): Include. (determine_invariantness_stmt): Record the loop a store is always executed in. * Makefile.in (tree-ssa-loop-im.o): Add tree-ssa-propagate.h dependency. * gcc.dg/tree-ssa/ssa-lim-5.c: New testcase. * gcc.dg/tree-ssa/ssa-lim-6.c: Likewise.. From-SVN: r135330
This commit is contained in:
parent
45ff5edaea
commit
8ded35f90c
@ -1,3 +1,13 @@
|
||||
2008-05-15 Richard Guenther <rguenther@suse.de>
|
||||
|
||||
PR tree-optimization/36009
|
||||
PR tree-optimization/36204
|
||||
* tree-ssa-loop-im.c (tree-ssa-propagate.h): Include.
|
||||
(determine_invariantness_stmt): Record the loop a store is
|
||||
always executed in.
|
||||
* Makefile.in (tree-ssa-loop-im.o): Add tree-ssa-propagate.h
|
||||
dependency.
|
||||
|
||||
2008-05-15 Richard Guenther <rguenther@suse.de>
|
||||
|
||||
PR tree-optimization/34330
|
||||
|
@ -2206,7 +2206,7 @@ tree-ssa-loop-im.o : tree-ssa-loop-im.c $(TREE_FLOW_H) $(CONFIG_H) \
|
||||
$(SYSTEM_H) $(RTL_H) $(TREE_H) $(TM_P_H) $(CFGLOOP_H) domwalk.h \
|
||||
$(PARAMS_H) output.h $(DIAGNOSTIC_H) $(TIMEVAR_H) $(TM_H) coretypes.h \
|
||||
$(TREE_DUMP_H) tree-pass.h $(FLAGS_H) $(REAL_H) $(BASIC_BLOCK_H) \
|
||||
hard-reg-set.h
|
||||
hard-reg-set.h tree-ssa-propagate.h
|
||||
tree-ssa-math-opts.o : tree-ssa-math-opts.c $(TREE_FLOW_H) $(CONFIG_H) \
|
||||
$(SYSTEM_H) $(TREE_H) $(TIMEVAR_H) tree-pass.h $(TM_H) $(FLAGS_H) \
|
||||
alloc-pool.h $(BASIC_BLOCK_H) $(TARGET_H)
|
||||
|
@ -1,3 +1,10 @@
|
||||
2008-05-15 Richard Guenther <rguenther@suse.de>
|
||||
|
||||
PR tree-optimization/36009
|
||||
PR tree-optimization/36204
|
||||
* gcc.dg/tree-ssa/ssa-lim-5.c: New testcase.
|
||||
* gcc.dg/tree-ssa/ssa-lim-6.c: Likewise..
|
||||
|
||||
2008-05-15 Richard Guenther <rguenther@suse.de>
|
||||
|
||||
PR tree-optimization/34330
|
||||
|
25
gcc/testsuite/gcc.dg/tree-ssa/ssa-lim-5.c
Normal file
25
gcc/testsuite/gcc.dg/tree-ssa/ssa-lim-5.c
Normal file
@ -0,0 +1,25 @@
|
||||
/* { dg-do link } */
|
||||
/* { dg-options "-O" } */
|
||||
|
||||
/* We should apply store motion here. */
|
||||
|
||||
struct BUF1
|
||||
{
|
||||
int b1;
|
||||
int b12;
|
||||
};
|
||||
|
||||
void link_error();
|
||||
|
||||
int foo(struct BUF1 * p)
|
||||
{
|
||||
|
||||
int i = 0;
|
||||
for (i = 0; i < 1024*1024; i++)
|
||||
p->b1 = 1;
|
||||
if (p->b1 != 1)
|
||||
link_error ();
|
||||
return 0;
|
||||
}
|
||||
|
||||
int main() { return 0; }
|
14
gcc/testsuite/gcc.dg/tree-ssa/ssa-lim-6.c
Normal file
14
gcc/testsuite/gcc.dg/tree-ssa/ssa-lim-6.c
Normal file
@ -0,0 +1,14 @@
|
||||
/* { dg-do compile } */
|
||||
/* { dg-options "-O2 -fdump-tree-lim-details" } */
|
||||
|
||||
double a[16][64], y[64], x[16];
|
||||
void foo(void)
|
||||
{
|
||||
int i, j;
|
||||
for (j = 0; j < 64; ++j)
|
||||
for (i = 0; i < 16; ++i)
|
||||
y[j] = y[j] + a[i][j] * x[i];
|
||||
}
|
||||
|
||||
/* { dg-final { scan-tree-dump "Executing store motion of y" "lim" } } */
|
||||
/* { dg-final { cleanup-tree-dump "lim" } } */
|
@ -40,6 +40,7 @@ along with GCC; see the file COPYING3. If not see
|
||||
#include "hashtab.h"
|
||||
#include "tree-affine.h"
|
||||
#include "pointer-set.h"
|
||||
#include "tree-ssa-propagate.h"
|
||||
|
||||
/* TODO: Support for predicated code motion. I.e.
|
||||
|
||||
@ -900,6 +901,14 @@ determine_invariantness_stmt (struct dom_walk_data *dw_data ATTRIBUTE_UNUSED,
|
||||
maybe_never = true;
|
||||
outermost = NULL;
|
||||
}
|
||||
/* Make sure to note always_executed_in for stores to make
|
||||
store-motion work. */
|
||||
else if (stmt_makes_single_store (stmt))
|
||||
{
|
||||
stmt_ann (stmt)->common.aux
|
||||
= xcalloc (1, sizeof (struct lim_aux_data));
|
||||
LIM_DATA (stmt)->always_executed_in = outermost;
|
||||
}
|
||||
continue;
|
||||
}
|
||||
|
||||
|
Loading…
Reference in New Issue
Block a user