diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 9602625ace4..8af0732838b 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2012-09-24 Richard Guenther + + PR middle-end/52173 + * gimple.c (gimple_copy): Properly mark the copy modified + if SSA operands are present. + 2012-09-23 Eric Botcazou PR tree-optimization/54669 diff --git a/gcc/gimple.c b/gcc/gimple.c index 7065040eb31..6088682e04c 100644 --- a/gcc/gimple.c +++ b/gcc/gimple.c @@ -2333,23 +2333,20 @@ gimple_copy (gimple stmt) } /* Make copy of operands. */ - if (num_ops > 0) + for (i = 0; i < num_ops; i++) + gimple_set_op (copy, i, unshare_expr (gimple_op (stmt, i))); + + if (gimple_has_mem_ops (stmt)) { - for (i = 0; i < num_ops; i++) - gimple_set_op (copy, i, unshare_expr (gimple_op (stmt, i))); + gimple_set_vdef (copy, gimple_vdef (stmt)); + gimple_set_vuse (copy, gimple_vuse (stmt)); + } - /* Clear out SSA operand vectors on COPY. */ - if (gimple_has_ops (stmt)) - { - gimple_set_def_ops (copy, NULL); - gimple_set_use_ops (copy, NULL); - } - - if (gimple_has_mem_ops (stmt)) - { - gimple_set_vdef (copy, gimple_vdef (stmt)); - gimple_set_vuse (copy, gimple_vuse (stmt)); - } + /* Clear out SSA operand vectors on COPY. */ + if (gimple_has_ops (stmt)) + { + gimple_set_def_ops (copy, NULL); + gimple_set_use_ops (copy, NULL); /* SSA operands need to be updated. */ gimple_set_modified (copy, true); diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index a677859668a..a2ea5084367 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,9 @@ +2012-09-24 Richard Guenther + + PR middle-end/52173 + * gcc.dg/tm/pr52173-1.c: New. + * gcc.dg/tm/pr52173-2.c: New. + 2012-09-23 Eric Botcazou * gcc.dg/pr54669.c: New test. diff --git a/gcc/testsuite/gcc.dg/tm/pr52173-1.c b/gcc/testsuite/gcc.dg/tm/pr52173-1.c new file mode 100644 index 00000000000..9ffa4d618a7 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tm/pr52173-1.c @@ -0,0 +1,19 @@ +/* { dg-do compile } */ +/* { dg-options "-fgnu-tm -O3" } */ + +int vec[500]; + +void func() +{ + __transaction_relaxed + { + vec[123] = 456; + } +} + +main() +{ + int i; + for(i = 0; i < 10; ++i) + func(); +} diff --git a/gcc/testsuite/gcc.dg/tm/pr52173-2.c b/gcc/testsuite/gcc.dg/tm/pr52173-2.c new file mode 100644 index 00000000000..ca64893fc06 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tm/pr52173-2.c @@ -0,0 +1,12 @@ +/* { dg-do compile } */ +/* { dg-options "-fgnu-tm -O2" } */ + +int a; + +int main() +{ + int i; + for (i = 0; i < 1; ++i) + __transaction_atomic { ++a; } + return 0; +}