From d9c259efd71ab62d4994b0ad489aa977f3ff736d Mon Sep 17 00:00:00 2001 From: Bin Cheng Date: Fri, 28 Jul 2017 15:01:28 +0000 Subject: [PATCH] tree-predcom.c: Revise general description of the pass. * tree-predcom.c: Revise general description of the pass. (enum chain_type): New enum type for store elimination. (struct chain): New field supporting store elimination. (struct component): Ditto. (dump_chain): Dump store-stores chain. (release_chain): Release resources. (split_data_refs_to_components): Compute and create component contains only stores for elimination. (get_chain_last_ref_at): New function. (make_invariant_chain): Initialization. (make_rooted_chain): Specify chain type in parameter and record it. (add_looparound_copies): Skip for store-stores chain. (determine_roots_comp): Compute type of chain and pass it to make_rooted_chain. (initialize_root_vars_store_elim_2): New function. (finalize_eliminated_stores): New function. (remove_stmt): Handle store for elimination. (execute_pred_commoning_chain): Execute predictive commoning on store-store chains. (determine_unroll_factor): Skip unroll for store-stores chain. (prepare_initializers_chain_store_elim): New function. (prepare_initializers_chain): Hanlde store-store chain. (prepare_finalizers_chain, prepare_finalizers): New function. (tree_predictive_commoning_loop): Return integer value indicating if loop is unrolled or lcssa form is corrupted. (tree_predictive_commoning): Rewrite for lcssa form if necessary. gcc/testsuite * gcc.dg/tree-ssa/predcom-dse-1.c: New test. * gcc.dg/tree-ssa/predcom-dse-2.c: New test. * gcc.dg/tree-ssa/predcom-dse-3.c: New test. * gcc.dg/tree-ssa/predcom-dse-4.c: New test. * gcc.dg/tree-ssa/predcom-dse-5.c: New test. * gcc.dg/tree-ssa/predcom-dse-6.c: New test. * gcc.dg/tree-ssa/predcom-dse-7.c: New test. * gcc.dg/tree-ssa/predcom-dse-8.c: New test. * gcc.dg/tree-ssa/predcom-dse-9.c: New test. * gcc.dg/tree-ssa/predcom-dse-10.c: New test. * gcc.dg/tree-ssa/predcom-dse-11.c: New test. From-SVN: r250669 --- gcc/ChangeLog | 29 ++ gcc/testsuite/ChangeLog | 14 + gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-1.c | 62 +++ .../gcc.dg/tree-ssa/predcom-dse-10.c | 44 ++ .../gcc.dg/tree-ssa/predcom-dse-11.c | 51 ++ gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-2.c | 62 +++ gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-3.c | 108 ++++ gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-4.c | 61 +++ gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-5.c | 63 +++ gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-6.c | 65 +++ gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-7.c | 63 +++ gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-8.c | 60 +++ gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-9.c | 90 ++++ gcc/tree-predcom.c | 480 ++++++++++++++++-- 14 files changed, 1206 insertions(+), 46 deletions(-) create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-1.c create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-10.c create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-11.c create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-2.c create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-3.c create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-4.c create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-5.c create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-6.c create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-7.c create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-8.c create mode 100644 gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-9.c diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 8af165486aa..fa27b96739a 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,32 @@ +2017-07-28 Bin Cheng + + * tree-predcom.c: Revise general description of the pass. + (enum chain_type): New enum type for store elimination. + (struct chain): New field supporting store elimination. + (struct component): Ditto. + (dump_chain): Dump store-stores chain. + (release_chain): Release resources. + (split_data_refs_to_components): Compute and create component + contains only stores for elimination. + (get_chain_last_ref_at): New function. + (make_invariant_chain): Initialization. + (make_rooted_chain): Specify chain type in parameter and record it. + (add_looparound_copies): Skip for store-stores chain. + (determine_roots_comp): Compute type of chain and pass it to + make_rooted_chain. + (initialize_root_vars_store_elim_2): New function. + (finalize_eliminated_stores): New function. + (remove_stmt): Handle store for elimination. + (execute_pred_commoning_chain): Execute predictive commoning on + store-store chains. + (determine_unroll_factor): Skip unroll for store-stores chain. + (prepare_initializers_chain_store_elim): New function. + (prepare_initializers_chain): Hanlde store-store chain. + (prepare_finalizers_chain, prepare_finalizers): New function. + (tree_predictive_commoning_loop): Return integer value indicating + if loop is unrolled or lcssa form is corrupted. + (tree_predictive_commoning): Rewrite for lcssa form if necessary. + 2017-07-28 Bin Cheng * tree-predcom.c (initialize_root): Delete. diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index b82e9bdd9be..e0c6bbec8c1 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,17 @@ +2017-07-28 Bin Cheng + + * gcc.dg/tree-ssa/predcom-dse-1.c: New test. + * gcc.dg/tree-ssa/predcom-dse-2.c: New test. + * gcc.dg/tree-ssa/predcom-dse-3.c: New test. + * gcc.dg/tree-ssa/predcom-dse-4.c: New test. + * gcc.dg/tree-ssa/predcom-dse-5.c: New test. + * gcc.dg/tree-ssa/predcom-dse-6.c: New test. + * gcc.dg/tree-ssa/predcom-dse-7.c: New test. + * gcc.dg/tree-ssa/predcom-dse-8.c: New test. + * gcc.dg/tree-ssa/predcom-dse-9.c: New test. + * gcc.dg/tree-ssa/predcom-dse-10.c: New test. + * gcc.dg/tree-ssa/predcom-dse-11.c: New test. + 2017-07-28 Martin Liska PR sanitizer/81340 diff --git a/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-1.c b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-1.c new file mode 100644 index 00000000000..d3a2339e55e --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-1.c @@ -0,0 +1,62 @@ +/* { dg-do run } */ +/* { dg-options "-O2 -fno-inline -fpredictive-commoning -fdump-tree-pcom-details" } */ + +int arr[105] = {2, 3, 5, 7, 11}; +int result0[10] = {2, 3, 5, 7, 11}; +int result1[10] = {0, -1, 5, -2, 11, 0}; +int result2[10] = {0, 0, -1, -2, -2, 0}; +int result3[10] = {0, 0, 0, -1, -2, -2, 0}; +int result4[10] = {0, 0, 0, 0, -1, -2, -2, 0}; +int result100[105] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1, -2, -2, 0}; + +extern void abort (void); + +void __attribute__((noinline)) foo (int *a, int len) +{ + int i; + for (i = 0; i < len; i++) + { + a[i] = 0; + a[i + 1] = -1; + a[i + 3] = -2; + } +} + +void check (int *a, int *res, int len) +{ + int i; + + for (i = 0; i < len; i++) + if (a[i] != res[i]) + abort (); +} + +int main (void) +{ + foo (arr, 0); + check (arr, result0, 10); + + foo (arr, 1); + check (arr, result1, 10); + + foo (arr, 2); + check (arr, result2, 10); + + foo (arr, 3); + check (arr, result3, 10); + + foo (arr, 4); + check (arr, result4, 10); + + foo (arr, 100); + check (arr, result100, 105); + + return 0; +} +/* { dg-final { scan-tree-dump "Store-stores chain" "pcom"} } */ diff --git a/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-10.c b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-10.c new file mode 100644 index 00000000000..bd5575d9502 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-10.c @@ -0,0 +1,44 @@ +/* { dg-do run } */ +/* { dg-options "-O2 -fno-inline -fpredictive-commoning -fdump-tree-pcom-details" } */ + +int arr[105] = {2, 3, 5, 7, 11}; +int result0[10] = {2, 3, 5, 7, 11}; +int result1[10] = {0, 3, 5, -2, 11, 0}; + +extern void abort (void); + +void __attribute__((noinline)) foo (int *a, int len, int flag) +{ + int i; + for (i = 0; i < len; i++) + { + a[i] = 0; + if (flag) + a[i + 1] = -1; + a[i + 3] = -2; + } +} + +void check (int *a, int *res, int len) +{ + int i; + + for (i = 0; i < len; i++) + if (a[i] != res[i]) + abort (); +} + +int main (void) +{ + int i; + + foo (arr, 0, 0); + check (arr, result0, 10); + + foo (arr, 1, 0); + check (arr, result1, 10); + + return 0; +} +/* { dg-final { scan-tree-dump-not "Store-stores chain" "pcom"} } */ + diff --git a/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-11.c b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-11.c new file mode 100644 index 00000000000..9e496f68a12 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-11.c @@ -0,0 +1,51 @@ +/* { dg-do run } */ +/* { dg-options "-O2 -fno-inline -fpredictive-commoning -fdump-tree-pcom-details" } */ + +int arr[105] = {2, 3, 5, 7, 11}; +int x[105] = {2, 3, 5, 7, 11}; +int result0[10] = {2, 3, 5, 7, 11}; +int result1[10] = {0, 0, 0, 0, 0, 0, -1, -2, -2}; + +extern void abort (void); + +int __attribute__((noinline)) foo (int * __restrict__ a, int * __restrict__ b, int len, int flag) +{ + int i, sum = 0; + for (i = 0; i < len; i++) + { + a[i] = 0; + b[i + 2] = i; + a[i + 1] = -1; + sum += b[i]; + a[i + 3] = -2; + } + return sum; +} + +void check (int *a, int *res, int len, int sum, int val) +{ + int i; + + if (sum != val) + abort (); + + for (i = 0; i < len; i++) + if (a[i] != res[i]) + abort (); +} + +int main (void) +{ + int i, sum; + + sum = foo (arr, x, 0, 0); + check (arr, result0, 10, sum, 0); + + sum = foo (arr, x, 6, 0); + check (arr, result1, 10, sum, 11); + + return 0; +} +/* { dg-final { scan-tree-dump "Store-stores chain" "pcom"} } */ +/* { dg-final { scan-tree-dump "Store-loads chain" "pcom"} } */ + diff --git a/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-2.c b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-2.c new file mode 100644 index 00000000000..c48d438e9e0 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-2.c @@ -0,0 +1,62 @@ +/* { dg-do run } */ +/* { dg-options "-O2 -fno-inline -fpredictive-commoning -fdump-tree-pcom-details" } */ + +int arr[105] = {2, 3, 5, 7, 11}; +int result0[10] = {2, 3, 5, 7, 11}; +int result1[10] = {0, 3, -1, -2, 11, 0}; +int result2[10] = {0, 0, -1, -1, -2, 0}; +int result3[10] = {0, 0, 0, -1, -1, -2, 0}; +int result4[10] = {0, 0, 0, 0, -1, -1, -2, 0}; +int result100[105] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1, -1, -2, 0}; + +extern void abort (void); + +void __attribute__((noinline)) foo (int *a, int len) +{ + int i; + for (i = 0; i < len; i++) + { + a[i] = 0; + a[i + 2] = -1; + a[i + 3] = -2; + } +} + +void check (int *a, int *res, int len) +{ + int i; + + for (i = 0; i < len; i++) + if (a[i] != res[i]) + abort (); +} + +int main (void) +{ + foo (arr, 0); + check (arr, result0, 10); + + foo (arr, 1); + check (arr, result1, 10); + + foo (arr, 2); + check (arr, result2, 10); + + foo (arr, 3); + check (arr, result3, 10); + + foo (arr, 4); + check (arr, result4, 10); + + foo (arr, 100); + check (arr, result100, 105); + + return 0; +} +/* { dg-final { scan-tree-dump "Store-stores chain" "pcom"} } */ diff --git a/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-3.c b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-3.c new file mode 100644 index 00000000000..9c2736cd0ad --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-3.c @@ -0,0 +1,108 @@ +/* { dg-do run } */ +/* { dg-options "-O2 -fno-inline -fpredictive-commoning -fdump-tree-pcom-details" } */ + +int arr1[105] = {2, 3, 5, 7, 11, 13, 0}; +int arr2[105] = {2, 3, 5, 7, 11, 13, 0}; +int arr3[105] = {2, 3, 5, 7, 11, 13, 0}; +int arr4[105] = {2, 3, 5, 7, 11, 13, 0}; +int result1[105] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1, -2, -3, 0}; +int result2[105] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1, -1, -2, 0}; +int result3[105] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1, -2, -2, 0}; +int result4[105] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1, -1, -1, 0}; + +extern void abort (void); + +void check (int *a, int *res, int len) +{ + int i; + + for (i = 0; i < len; i++) + if (a[i] != res[i]) + abort (); +} + +void __attribute__((noinline)) foo1 (int *a) +{ + int i; + for (i = 0; i < 100; i++) + { + a[i] = 0; + a[i + 1] = -1; + a[i + 2] = -2; + a[i + 3] = -3; + } +} + +void __attribute__((noinline)) foo2 (int *a) +{ + int i; + for (i = 0; i < 100; i++) + { + a[i] = 0; + a[i + 2] = -1; + a[i + 3] = -2; + } +} + +void __attribute__((noinline)) foo3 (int *a) +{ + int i; + for (i = 0; i < 100; i++) + { + a[i] = 0; + a[i + 1] = -1; + a[i + 3] = -2; + } +} + +void __attribute__((noinline)) foo4 (int *a) +{ + int i; + for (i = 0; i < 100; i++) + { + a[i] = 0; + a[i + 3] = -1; + } +} + +int main (void) +{ + foo1 (arr1); + check (arr1, result1, 105); + + foo2 (arr2); + check (arr2, result2, 105); + + foo3 (arr3); + check (arr3, result3, 105); + + foo4 (arr4); + check (arr4, result4, 105); + + return 0; +} +/* { dg-final { scan-tree-dump-times "Store-stores chain" 4 "pcom"} } */ diff --git a/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-4.c b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-4.c new file mode 100644 index 00000000000..302425a17cc --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-4.c @@ -0,0 +1,61 @@ +/* { dg-do run } */ +/* { dg-options "-O2 -fno-inline -fpredictive-commoning -fdump-tree-pcom-details" } */ + +int arr[105] = {2, 3, 5, 7, 11}; +int result0[10] = {2, 3, 5, 7, 11}; +int result1[10] = {0, 3, 5, -1, 11, 0}; +int result2[10] = {0, 0, 5, -1, -1, 0}; +int result3[10] = {0, 0, 0, -1, -1, -1, 0}; +int result4[10] = {0, 0, 0, 0, -1, -1, -1, 0}; +int result100[105] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1, -1, -1, 0}; + +extern void abort (void); + +void __attribute__((noinline)) foo (int *a, int len) +{ + int i; + for (i = 0; i < len; i++) + { + a[i] = 0; + a[i + 3] = -1; + } +} + +void check (int *a, int *res, int len) +{ + int i; + + for (i = 0; i < len; i++) + if (a[i] != res[i]) + abort (); +} + +int main (void) +{ + foo (arr, 0); + check (arr, result0, 10); + + foo (arr, 1); + check (arr, result1, 10); + + foo (arr, 2); + check (arr, result2, 10); + + foo (arr, 3); + check (arr, result3, 10); + + foo (arr, 4); + check (arr, result4, 10); + + foo (arr, 100); + check (arr, result100, 105); + + return 0; +} +/* { dg-final { scan-tree-dump "Store-stores chain" "pcom"} } */ diff --git a/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-5.c b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-5.c new file mode 100644 index 00000000000..a13d56098be --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-5.c @@ -0,0 +1,63 @@ +/* { dg-do run } */ +/* { dg-options "-O2 -fno-inline -fpredictive-commoning -fdump-tree-pcom-details" } */ + +int arr[105] = {2, 3, 5, 7, 11}; +int result0[10] = {2, 3, 5, 7, 11}; +int result1[10] = {0, -1, -2, -3, 11, 0}; +int result2[10] = {0, 0, -1, -2, -3, 0}; +int result3[10] = {0, 0, 0, -1, -2, -3, 0}; +int result4[10] = {0, 0, 0, 0, -1, -2, -3, 0}; +int result100[105] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1, -2, -3, 0}; + +extern void abort (void); + +void __attribute__((noinline)) foo (int *a, int len) +{ + int i; + for (i = 0; i < len; i++) + { + a[i] = 0; + a[i + 1] = -1; + a[i + 2] = -2; + a[i + 3] = -3; + } +} + +void check (int *a, int *res, int len) +{ + int i; + + for (i = 0; i < len; i++) + if (a[i] != res[i]) + abort (); +} + +int main (void) +{ + foo (arr, 0); + check (arr, result0, 10); + + foo (arr, 1); + check (arr, result1, 10); + + foo (arr, 2); + check (arr, result2, 10); + + foo (arr, 3); + check (arr, result3, 10); + + foo (arr, 4); + check (arr, result4, 10); + + foo (arr, 100); + check (arr, result100, 105); + + return 0; +} +/* { dg-final { scan-tree-dump "Store-stores chain" "pcom"} } */ diff --git a/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-6.c b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-6.c new file mode 100644 index 00000000000..63d6c8f33b0 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-6.c @@ -0,0 +1,65 @@ +/* { dg-do run } */ +/* { dg-options "-O2 -fno-inline -fpredictive-commoning -fdump-tree-pcom-details" } */ + +int arr[105] = {2, 3, 5, 7, 11, 13, 17, 19}; +int result0[10] = {2, 3, 5, 7, 11, 13, 17, 19}; +int result1[10] = {0, -1, -2, -3, -4, -5, 17, 19}; +int result2[10] = {0, 0, -1, -2, -3, -4, -5, 19}; +int result3[10] = {0, 0, 0, -1, -2, -3, -4, -5}; +int result4[10] = {0, 0, 0, 0, -1, -2, -3, -4, -5}; +int result100[105] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1, -2, -3, -4, -5}; + +extern void abort (void); + +void __attribute__((noinline)) foo (int *a, int len) +{ + int i; + for (i = 0; i < len; i++) + { + a[i] = 0; + a[i + 1] = -1; + a[i + 2] = -2; + a[i + 3] = -3; + a[i + 4] = -4; + a[i + 5] = -5; + } +} + +void check (int *a, int *res, int len) +{ + int i; + + for (i = 0; i < len; i++) + if (a[i] != res[i]) + abort (); +} + +int main (void) +{ + foo (arr, 0); + check (arr, result0, 10); + + foo (arr, 1); + check (arr, result1, 10); + + foo (arr, 2); + check (arr, result2, 10); + + foo (arr, 3); + check (arr, result3, 10); + + foo (arr, 4); + check (arr, result4, 10); + + foo (arr, 100); + check (arr, result100, 105); + + return 0; +} +/* { dg-final { scan-tree-dump "Store-stores chain" "pcom"} } */ diff --git a/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-7.c b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-7.c new file mode 100644 index 00000000000..0bde6e6dced --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-7.c @@ -0,0 +1,63 @@ +/* { dg-do run } */ +/* { dg-options "-O2 -fno-inline -fpredictive-commoning -fdump-tree-pcom-details" } */ + +int arr[105] = {2, 3, 5, 7, 11, 13, 17, 19}; +int result0[10] = {2, 3, 5, 7, 11, 13, 17, 19}; +int result1[10] = {0, -1, 5, -3, 11, -5, 17, 19}; +int result2[10] = {0, 0, -1, -3, -3, -5, -5, 19}; +int result3[10] = {0, 0, 0, -1, -3, -3, -5, -5}; +int result4[10] = {0, 0, 0, 0, -1, -3, -3, -5, -5}; +int result100[105] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1, -3, -3, -5, -5}; + +extern void abort (void); + +void __attribute__((noinline)) foo (int *a, int len) +{ + int i; + for (i = 0; i < len; i++) + { + a[i] = 0; + a[i + 1] = -1; + a[i + 3] = -3; + a[i + 5] = -5; + } +} + +void check (int *a, int *res, int len) +{ + int i; + + for (i = 0; i < len; i++) + if (a[i] != res[i]) + abort (); +} + +int main (void) +{ + foo (arr, 0); + check (arr, result0, 10); + + foo (arr, 1); + check (arr, result1, 10); + + foo (arr, 2); + check (arr, result2, 10); + + foo (arr, 3); + check (arr, result3, 10); + + foo (arr, 4); + check (arr, result4, 10); + + foo (arr, 100); + check (arr, result100, 105); + + return 0; +} +/* { dg-final { scan-tree-dump "Store-stores chain" "pcom"} } */ diff --git a/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-8.c b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-8.c new file mode 100644 index 00000000000..45ffd25c424 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-8.c @@ -0,0 +1,60 @@ +/* { dg-do run } */ +/* { dg-options "-O2 -fno-inline -fpredictive-commoning -fdump-tree-pcom-details" } */ + +int arr[105] = {2, 3, 5, 7, 11, 13, 17, 19}; +int result0[10] = {2, 3, 5, 7, 11, 13, 17, 19}; +int result1[10] = {0, 3, 5, 7, 11, -5, 17, 19}; +int result2[10] = {0, 0, 5, 7, 11, -5, -5, 19}; +int result3[10] = {0, 0, 0, 7, 11, -5, -5, -5}; +int result4[10] = {0, 0, 0, 0, 11, -5, -5, -5, -5}; +int result100[105] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -5, -5, -5, -5, -5}; + +extern void abort (void); + +void __attribute__((noinline)) foo (int *a, int len) +{ + int i; + for (i = 0; i < len; i++) + { + a[i] = 0; + a[i + 5] = -5; + } +} + +void check (int *a, int *res, int len) +{ + int i; + + for (i = 0; i < len; i++) + if (a[i] != res[i]) + abort (); +} + +int main (void) +{ + foo (arr, 0); + check (arr, result0, 10); + + foo (arr, 1); + check (arr, result1, 10); + + foo (arr, 2); + check (arr, result2, 10); + + foo (arr, 3); + check (arr, result3, 10); + + foo (arr, 4); + check (arr, result4, 10); + + foo (arr, 100); + check (arr, result100, 105); + + return 0; +} diff --git a/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-9.c b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-9.c new file mode 100644 index 00000000000..1c4e3140309 --- /dev/null +++ b/gcc/testsuite/gcc.dg/tree-ssa/predcom-dse-9.c @@ -0,0 +1,90 @@ +/* { dg-do run } */ +/* { dg-options "-O2 -fno-inline -fpredictive-commoning -fdump-tree-pcom-details" } */ + +int arr1[105] = {2, 3, 5, 7, 11, 13, 17, 19}; +int arr2[105] = {2, 3, 5, 7, 11, 13, 17, 19}; +int arr3[105] = {2, 3, 5, 7, 11, 13, 17, 19}; + +int result1[105] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1, -2, -3, -4, -5}; +int result2[105] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -1, -3, -3, -5, -5}; +int result3[105] = {0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, + 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, -5, -5, -5, -5, -5}; + +extern void abort (void); + +void __attribute__((noinline)) foo1 (int *a) +{ + int i; + for (i = 0; i < 100; i++) + { + a[i] = 0; + a[i + 1] = -1; + a[i + 2] = -2; + a[i + 3] = -3; + a[i + 4] = -4; + a[i + 5] = -5; + } +} + +void __attribute__((noinline)) foo2 (int *a) +{ + int i; + for (i = 0; i < 100; i++) + { + a[i] = 0; + a[i + 1] = -1; + a[i + 3] = -3; + a[i + 5] = -5; + } +} + +void __attribute__((noinline)) foo3 (int *a) +{ + int i; + for (i = 0; i < 100; i++) + { + a[i] = 0; + a[i + 5] = -5; + } +} + +void check (int *a, int *res, int len) +{ + int i; + + for (i = 0; i < len; i++) + if (a[i] != res[i]) + abort (); +} + +int main (void) +{ + foo1 (arr1); + check (arr1, result1, 10); + + foo2 (arr2); + check (arr2, result2, 10); + + foo3 (arr3); + check (arr3, result3, 10); + + return 0; +} +/* { dg-final { scan-tree-dump "Store-stores chain" "pcom"} } */ diff --git a/gcc/tree-predcom.c b/gcc/tree-predcom.c index f8757437897..ac0d0d09484 100644 --- a/gcc/tree-predcom.c +++ b/gcc/tree-predcom.c @@ -156,29 +156,45 @@ along with GCC; see the file COPYING3. If not see b[10] = x; } - TODO -- stores killing other stores can be taken into account, e.g., - for (i = 0; i < n; i++) - { - a[i] = 1; - a[i+2] = 2; - } + Apart from predictive commoning on Load-Load and Store-Load chains, we + also support Store-Store chains -- stores killed by other store can be + eliminated. Given below example: + + for (i = 0; i < n; i++) + { + a[i] = 1; + a[i+2] = 2; + } - can be replaced with + It can be replaced with: - t0 = a[0]; - t1 = a[1]; - for (i = 0; i < n; i++) - { - a[i] = 1; - t2 = 2; - t0 = t1; - t1 = t2; - } - a[n] = t0; - a[n+1] = t1; + t0 = a[0]; + t1 = a[1]; + for (i = 0; i < n; i++) + { + a[i] = 1; + t2 = 2; + t0 = t1; + t1 = t2; + } + a[n] = t0; + a[n+1] = t1; - The interesting part is that this would generalize store motion; still, since - sm is performed elsewhere, it does not seem that important. + If the loop runs more than 1 iterations, it can be further simplified into: + + for (i = 0; i < n; i++) + { + a[i] = 1; + } + a[n] = 2; + a[n+1] = 2; + + The interesting part is this can be viewed either as general store motion + or general dead store elimination in either intra/inter-iterations way. + + TODO: For now, we don't support store-store chains in multi-exit loops. We + force to not unroll in case of store-store chain even if other chains might + ask for unroll. Predictive commoning can be generalized for arbitrary computations (not just memory loads), and also nontrivial transfer functions (e.g., replacing @@ -265,6 +281,9 @@ enum chain_type /* Root of the chain is store, the rest are loads. */ CT_STORE_LOAD, + /* There are only stores in the chain. */ + CT_STORE_STORE, + /* A combination of two chains. */ CT_COMBINATION }; @@ -294,9 +313,15 @@ typedef struct chain /* Initializers for the variables. */ vec inits; + /* Finalizers for the eliminated stores. */ + vec finis; + /* gimple stmts intializing the initial variables of the chain. */ gimple_seq init_seq; + /* gimple stmts finalizing the eliminated stores of the chain. */ + gimple_seq fini_seq; + /* True if there is a use of a variable with the maximal distance that comes after the root in the loop. */ unsigned has_max_use_after : 1; @@ -334,6 +359,10 @@ struct component /* What we know about the step of the references in the component. */ enum ref_step_type comp_step; + /* True if all references in component are stores and we try to do + intra/inter loop iteration dead store elimination. */ + bool eliminate_store_p; + /* Next component in the list. */ struct component *next; }; @@ -404,6 +433,10 @@ dump_chain (FILE *file, chain_p chain) chain_type = "Store-loads"; break; + case CT_STORE_STORE: + chain_type = "Store-stores"; + break; + case CT_COMBINATION: chain_type = "Combination"; break; @@ -517,6 +550,10 @@ release_chain (chain_p chain) if (chain->init_seq) gimple_seq_discard (chain->init_seq); + chain->finis.release (); + if (chain->fini_seq) + gimple_seq_discard (chain->fini_seq); + free (chain); } @@ -719,6 +756,8 @@ split_data_refs_to_components (struct loop *loop, struct data_dependence_relation *ddr; struct component *comp_list = NULL, *comp; dref dataref; + /* Don't do store elimination if loop has multiple exit edges. */ + bool eliminate_store_p = single_exit (loop) != NULL; basic_block last_always_executed = last_always_executed_block (loop); FOR_EACH_VEC_ELT (datarefs, i, dr) @@ -761,6 +800,14 @@ split_data_refs_to_components (struct loop *loop, dra = DDR_A (ddr); drb = DDR_B (ddr); + + /* Don't do store elimination if there is any unknown dependence for + any store data reference. */ + if ((DR_IS_WRITE (dra) || DR_IS_WRITE (drb)) + && (DDR_ARE_DEPENDENT (ddr) == chrec_dont_know + || DDR_NUM_DIST_VECTS (ddr) == 0)) + eliminate_store_p = false; + ia = component_of (comp_father, (unsigned) (size_t) dra->aux); ib = component_of (comp_father, (unsigned) (size_t) drb->aux); if (ia == ib) @@ -800,10 +847,28 @@ split_data_refs_to_components (struct loop *loop, continue; } } + else if (DR_IS_WRITE (dra) && DR_IS_WRITE (drb) + && ia != bad && ib != bad + && !determine_offset (dra, drb, &dummy_off)) + { + merge_comps (comp_father, comp_size, bad, ia); + merge_comps (comp_father, comp_size, bad, ib); + continue; + } merge_comps (comp_father, comp_size, ia, ib); } + if (eliminate_store_p) + { + tree niters = number_of_latch_executions (loop); + + /* Don't do store elimination if niters info is unknown because stores + in the last iteration can't be eliminated and we need to recover it + after loop. */ + eliminate_store_p = (niters != NULL_TREE && niters != chrec_dont_know); + } + comps = XCNEWVEC (struct component *, n); bad = component_of (comp_father, n); FOR_EACH_VEC_ELT (datarefs, i, dr) @@ -818,6 +883,7 @@ split_data_refs_to_components (struct loop *loop, { comp = XCNEW (struct component); comp->refs.create (comp_size[ca]); + comp->eliminate_store_p = eliminate_store_p; comps[ca] = comp; } @@ -832,6 +898,8 @@ split_data_refs_to_components (struct loop *loop, gimple_bb (dataref->stmt)); dataref->pos = comp->refs.length (); comp->refs.quick_push (dataref); + if (DR_IS_READ (dr)) + comp->eliminate_store_p = false; } for (i = 0; i < n; i++) @@ -956,6 +1024,21 @@ get_chain_root (chain_p chain) return chain->refs[0]; } +/* Given CHAIN, returns the last ref at DISTANCE, or NULL if it doesn't + exist. */ + +static inline dref +get_chain_last_ref_at (chain_p chain, unsigned distance) +{ + unsigned i; + + for (i = chain->refs.length (); i > 0; i--) + if (distance == chain->refs[i - 1]->distance) + break; + + return (i > 0) ? chain->refs[i - 1] : NULL; +} + /* Adds REF to the chain CHAIN. */ static void @@ -1008,23 +1091,27 @@ make_invariant_chain (struct component *comp) chain->all_always_accessed &= ref->always_accessed; } + chain->inits = vNULL; + chain->finis = vNULL; + return chain; } -/* Make a new chain rooted at REF. */ +/* Make a new chain of type TYPE rooted at REF. */ static chain_p -make_rooted_chain (dref ref) +make_rooted_chain (dref ref, enum chain_type type) { chain_p chain = XCNEW (struct chain); - chain->type = DR_IS_READ (ref->ref) ? CT_LOAD : CT_STORE_LOAD; - + chain->type = type; chain->refs.safe_push (ref); chain->all_always_accessed = ref->always_accessed; - ref->distance = 0; + chain->inits = vNULL; + chain->finis = vNULL; + return chain; } @@ -1199,6 +1286,9 @@ add_looparound_copies (struct loop *loop, chain_p chain) dref ref, root = get_chain_root (chain); gphi *phi; + if (chain->type == CT_STORE_STORE) + return; + FOR_EACH_VEC_ELT (chain->refs, i, ref) { phi = find_looparound_phi (loop, ref, root); @@ -1223,6 +1313,7 @@ determine_roots_comp (struct loop *loop, dref a; chain_p chain = NULL; widest_int last_ofs = 0; + enum chain_type type; /* Invariants are handled specially. */ if (comp->comp_step == RS_INVARIANT) @@ -1239,7 +1330,8 @@ determine_roots_comp (struct loop *loop, comp->refs.qsort (order_drefs); FOR_EACH_VEC_ELT (comp->refs, i, a) { - if (!chain || DR_IS_WRITE (a->ref) + if (!chain + || (!comp->eliminate_store_p && DR_IS_WRITE (a->ref)) || wi::leu_p (MAX_DISTANCE, a->offset - last_ofs)) { if (nontrivial_chain_p (chain)) @@ -1249,7 +1341,13 @@ determine_roots_comp (struct loop *loop, } else release_chain (chain); - chain = make_rooted_chain (a); + + if (DR_IS_READ (a->ref)) + type = CT_LOAD; + else + type = comp->eliminate_store_p ? CT_STORE_STORE : CT_STORE_LOAD; + + chain = make_rooted_chain (a, type); last_ofs = a->offset; continue; } @@ -1536,6 +1634,114 @@ initialize_root_vars (struct loop *loop, chain_p chain, bitmap tmp_vars) } } +/* Creates root variables for store elimination CHAIN in which stores for + elimination store loop variant values. In this case, we may need to + load root variables before LOOP and propagate it with PHI nodes. Uids + of the newly created root variables are marked in TMP_VARS. */ + +static void +initialize_root_vars_store_elim_2 (struct loop *loop, + chain_p chain, bitmap tmp_vars) +{ + unsigned i, n = chain->length; + tree ref, init, var, next, val, phi_result; + gimple *stmt; + gimple_seq stmts; + + chain->vars.create (n); + + ref = DR_REF (get_chain_root (chain)->ref); + for (i = 0; i < n; i++) + { + var = predcom_tmp_var (ref, i, tmp_vars); + chain->vars.quick_push (var); + } + + FOR_EACH_VEC_ELT (chain->vars, i, var) + chain->vars[i] = make_ssa_name (var); + + /* Root values are either rhs operand of stores to be eliminated, or + loaded from memory before loop. */ + auto_vec vtemps; + vtemps.safe_grow_cleared (n); + for (i = 0; i < n; i++) + { + init = get_init_expr (chain, i); + if (init == NULL_TREE) + { + /* Root value is rhs operand of the store to be eliminated if + it isn't loaded from memory before loop. */ + dref a = get_chain_last_ref_at (chain, i); + val = gimple_assign_rhs1 (a->stmt); + if (TREE_CLOBBER_P (val)) + val = get_or_create_ssa_default_def (cfun, SSA_NAME_VAR (var)); + + vtemps[n - i - 1] = val; + } + else + { + edge latch = loop_latch_edge (loop); + edge entry = loop_preheader_edge (loop); + + /* Root value is loaded from memory before loop, we also need + to add PHI nodes to propagate the value across iterations. */ + init = force_gimple_operand (init, &stmts, true, NULL_TREE); + if (stmts) + gsi_insert_seq_on_edge_immediate (entry, stmts); + + next = chain->vars[n - i]; + phi_result = copy_ssa_name (next); + gphi *phi = create_phi_node (phi_result, loop->header); + add_phi_arg (phi, init, entry, UNKNOWN_LOCATION); + add_phi_arg (phi, next, latch, UNKNOWN_LOCATION); + vtemps[n - i - 1] = phi_result; + } + } + + /* Find the insertion position. */ + dref last = get_chain_root (chain); + for (i = 0; i < chain->refs.length (); i++) + { + if (chain->refs[i]->pos > last->pos) + last = chain->refs[i]; + } + + gimple_stmt_iterator gsi = gsi_for_stmt (last->stmt); + + /* Insert statements copying root value to root variable. */ + for (i = 0; i < n; i++) + { + var = chain->vars[i]; + val = vtemps[i]; + stmt = gimple_build_assign (var, val); + gsi_insert_after (&gsi, stmt, GSI_NEW_STMT); + } +} + +/* Generates stores for CHAIN's eliminated stores in LOOP's last + (CHAIN->length - 1) iterations. */ + +static void +finalize_eliminated_stores (struct loop *loop, chain_p chain) +{ + unsigned i, n = chain->length; + + for (i = 0; i < n; i++) + { + tree var = chain->vars[i]; + tree fini = chain->finis[n - i - 1]; + gimple *stmt = gimple_build_assign (fini, var); + + gimple_seq_add_stmt_without_update (&chain->fini_seq, stmt); + } + + if (chain->fini_seq) + { + gsi_insert_seq_on_edge_immediate (single_exit (loop), chain->fini_seq); + chain->fini_seq = NULL; + } +} + /* Initializes a variable for load motion for ROOT and prepares phi nodes and initialization on entry to LOOP if necessary. The ssa name for the variable is stored in VARS. If WRITTEN is true, also a phi node to copy its value @@ -1704,10 +1910,17 @@ remove_stmt (gimple *stmt) bsi = gsi_for_stmt (stmt); name = gimple_assign_lhs (stmt); - gcc_assert (TREE_CODE (name) == SSA_NAME); - - next = single_nonlooparound_use (name); - reset_debug_uses (stmt); + if (TREE_CODE (name) == SSA_NAME) + { + next = single_nonlooparound_use (name); + reset_debug_uses (stmt); + } + else + { + /* This is a store to be eliminated. */ + gcc_assert (gimple_vdef (stmt) != NULL); + next = NULL; + } unlink_stmt_vdef (stmt); gsi_remove (&bsi, true); @@ -1727,9 +1940,9 @@ remove_stmt (gimple *stmt) static void execute_pred_commoning_chain (struct loop *loop, chain_p chain, - bitmap tmp_vars) + bitmap tmp_vars) { - unsigned i; + unsigned i, n; dref a; tree var; bool in_lhs; @@ -1740,6 +1953,27 @@ execute_pred_commoning_chain (struct loop *loop, chain_p chain, compute the values of the expression (except for the root one). We delay this until after all chains are processed. */ } + else if (chain->type == CT_STORE_STORE) + { + if (chain->length > 0) + { + /* For inter-iteration store elimination chain, set up the + variables by loading from memory before loop, copying from rhs + of stores for elimination and propagate it with PHI nodes. */ + initialize_root_vars_store_elim_2 (loop, chain, tmp_vars); + + /* For inter-iteration store elimination chain, stores at each + distance in loop's last (chain->length - 1) iterations can't + be eliminated, because there is no following killing store. + We need to generate these stores after loop. */ + finalize_eliminated_stores (loop, chain); + } + + /* Eliminate the stores killed by following store. */ + n = chain->refs.length (); + for (i = 0; i < n - 1; i++) + remove_stmt (chain->refs[i]->stmt); + } else { /* For non-combined chains, set up the variables that hold its value. */ @@ -1773,6 +2007,9 @@ determine_unroll_factor (vec chains) { if (chain->type == CT_INVARIANT) continue; + /* For now we can't handle unrolling when eliminating stores. */ + else if (chain->type == CT_STORE_STORE) + return 1; if (chain->combined) { @@ -2419,6 +2656,65 @@ try_combine_chains (vec *chains) } } +/* Prepare initializers for store elimination CHAIN in LOOP. Returns false + if this is impossible because one of these initializers may trap, true + otherwise. */ + +static bool +prepare_initializers_chain_store_elim (struct loop *, chain_p chain) +{ + unsigned i, n = chain->length; + + /* For now we can't eliminate stores if some of them are conditional + executed. */ + if (!chain->all_always_accessed) + return false; + + /* Nothing to intialize for intra-iteration store elimination. */ + if (n == 0 && chain->type == CT_STORE_STORE) + return true; + + chain->inits.create (n); + chain->inits.safe_grow_cleared (n); + + /* For store eliminatin chain like below: + + for (i = 0; i < len; i++) + { + a[i] = 1; + // a[i + 1] = ... + a[i + 2] = 3; + } + + store to a[i + 1] is missed in loop body, it acts like bubbles. The + content of a[i + 1] remain the same if the loop iterates fewer times + than chain->length. We need to set up root variables for such stores + by loading from memory before loop. Note we only need to load bubble + elements because loop body is guaranteed to be executed at least once + after loop's preheader edge. */ + auto_vec bubbles; + bubbles.safe_grow_cleared (n + 1); + for (i = 0; i < chain->refs.length (); i++) + bubbles[chain->refs[i]->distance] = true; + + struct data_reference *dr = get_chain_root (chain)->ref; + for (i = 0; i < n; i++) + { + if (bubbles[i]) + continue; + + gimple_seq stmts = NULL; + + tree init = ref_at_iteration (dr, (int) 0 - i, &stmts); + if (stmts) + gimple_seq_add_seq_without_update (&chain->init_seq, stmts); + + chain->inits[i] = init; + } + + return true; +} + /* Prepare initializers for CHAIN in LOOP. Returns false if this is impossible because one of these initializers may trap, true otherwise. */ @@ -2431,6 +2727,9 @@ prepare_initializers_chain (struct loop *loop, chain_p chain) dref laref; edge entry = loop_preheader_edge (loop); + if (chain->type == CT_STORE_STORE) + return prepare_initializers_chain_store_elim (loop, chain); + /* Find the initializers for the variables, and check that they cannot trap. */ chain->inits.create (n); @@ -2494,6 +2793,90 @@ prepare_initializers (struct loop *loop, vec chains) } } +/* Generates finalizer memory references for CHAIN in LOOP. Returns true + if finalizer code for CHAIN can be generated, otherwise false. */ + +static bool +prepare_finalizers_chain (struct loop *loop, chain_p chain) +{ + unsigned i, n = chain->length; + struct data_reference *dr = get_chain_root (chain)->ref; + tree fini, niters = number_of_latch_executions (loop); + + /* For now we can't eliminate stores if some of them are conditional + executed. */ + if (!chain->all_always_accessed) + return false; + + chain->finis.create (n); + for (i = 0; i < n; i++) + chain->finis.quick_push (NULL_TREE); + + /* We never use looparound phi node for store elimination chains. */ + + /* Find the finalizers for the variables, and check that they cannot + trap. */ + for (i = 0; i < n; i++) + { + gimple_seq stmts = NULL; + gcc_assert (chain->finis[i] == NULL_TREE); + + if (TREE_CODE (niters) != INTEGER_CST && TREE_CODE (niters) != SSA_NAME) + { + niters = copy_node (niters); + niters = force_gimple_operand (niters, &stmts, true, NULL); + if (stmts) + { + gimple_seq_add_seq_without_update (&chain->fini_seq, stmts); + stmts = NULL; + } + } + fini = ref_at_iteration (dr, (int) 0 - i, &stmts, niters); + if (stmts) + gimple_seq_add_seq_without_update (&chain->fini_seq, stmts); + + chain->finis[i] = fini; + } + + return true; +} + +/* Generates finalizer memory reference for CHAINS in LOOP. Returns true + if finalizer code generation for CHAINS breaks loop closed ssa form. */ + +static bool +prepare_finalizers (struct loop *loop, vec chains) +{ + chain_p chain; + unsigned i; + bool loop_closed_ssa = false; + + for (i = 0; i < chains.length ();) + { + chain = chains[i]; + + /* Finalizer is only necessary for inter-iteration store elimination + chains. */ + if (chain->length == 0 || chain->type != CT_STORE_STORE) + { + i++; + continue; + } + + if (prepare_finalizers_chain (loop, chain)) + { + i++; + loop_closed_ssa = true; + } + else + { + release_chain (chain); + chains.unordered_remove (i); + } + } + return loop_closed_ssa; +} + /* Insert all initializing gimple stmts into loop's entry edge. */ static void @@ -2510,10 +2893,11 @@ insert_init_seqs (struct loop *loop, vec chains) } } -/* Performs predictive commoning for LOOP. Returns true if LOOP was - unrolled. */ +/* Performs predictive commoning for LOOP. Sets bit 1<<0 of return value + if LOOP was unrolled; Sets bit 1<<1 of return value if loop closed ssa + form was corrupted. */ -static bool +static unsigned tree_predictive_commoning_loop (struct loop *loop) { vec datarefs; @@ -2522,7 +2906,7 @@ tree_predictive_commoning_loop (struct loop *loop) vec chains = vNULL; unsigned unroll_factor; struct tree_niter_desc desc; - bool unroll = false; + bool unroll = false, loop_closed_ssa = false; edge exit; if (dump_file && (dump_flags & TDF_DETAILS)) @@ -2534,7 +2918,7 @@ tree_predictive_commoning_loop (struct loop *loop) if (dump_file && (dump_flags & TDF_DETAILS)) fprintf (dump_file, "Loop iterates only 1 time, nothing to do.\n"); - return false; + return 0; } /* Find the data references and split them into components according to their @@ -2549,7 +2933,7 @@ tree_predictive_commoning_loop (struct loop *loop) fprintf (dump_file, "Cannot analyze data dependencies\n"); free_data_refs (datarefs); free_dependence_relations (dependences); - return false; + return 0; } if (dump_file && (dump_flags & TDF_DETAILS)) @@ -2562,7 +2946,7 @@ tree_predictive_commoning_loop (struct loop *loop) { free_data_refs (datarefs); free_affine_expand_cache (&name_expansions); - return false; + return 0; } if (dump_file && (dump_flags & TDF_DETAILS)) @@ -2587,6 +2971,7 @@ tree_predictive_commoning_loop (struct loop *loop) goto end; } prepare_initializers (loop, chains); + loop_closed_ssa = prepare_finalizers (loop, chains); /* Try to combine the chains that are always worked with together. */ try_combine_chains (&chains); @@ -2648,7 +3033,7 @@ end: ; free_affine_expand_cache (&name_expansions); - return unroll; + return (unroll ? 1 : 0) | (loop_closed_ssa ? 2 : 0); } /* Runs predictive commoning. */ @@ -2656,23 +3041,26 @@ end: ; unsigned tree_predictive_commoning (void) { - bool unrolled = false; struct loop *loop; - unsigned ret = 0; + unsigned ret = 0, changed = 0; initialize_original_copy_tables (); FOR_EACH_LOOP (loop, LI_ONLY_INNERMOST) if (optimize_loop_for_speed_p (loop)) { - unrolled |= tree_predictive_commoning_loop (loop); + changed |= tree_predictive_commoning_loop (loop); } + free_original_copy_tables (); - if (unrolled) + if (changed > 0) { scev_reset (); + + if (changed > 1) + rewrite_into_loop_closed_ssa (NULL, TODO_update_ssa); + ret = TODO_cleanup_cfg; } - free_original_copy_tables (); return ret; } -- 2.30.2