[PATCH GCC][3/5]Move pass ivcanon upward in compilation process

[PATCH GCC][3/5]Move pass ivcanon upward in compilation process

[Bin Cheng]

[-- Attachment #1: Type: text/plain, Size: 403 bytes --]

Hi,
This patch moves pass ivcanon before loop distribution.  Pass loop split could create
loops with limited niters.  Such loop should be unrolled before loop distribution (or graphite),
rather than after.

Bootstrap and test at O2/O3 on x86_64 and AArch64.  is it OK?

Thanks,
bin
2017-05-31  Bin Cheng  <bin.cheng@arm.com>

	* passes.def (pass_iv_canon): Move before pass_loop_distribution.

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0003-move-ivcanon-pass-20170529.txt.patch --]
[-- Type: text/x-patch; name="0003-move-ivcanon-pass-20170529.txt.patch", Size: 1062 bytes --]

From 1698cc3e552a17e84719dba1ff2fbe4a8890e6be Mon Sep 17 00:00:00 2001
From: Bin Cheng <binche01@e108451-lin.cambridge.arm.com>
Date: Tue, 30 May 2017 17:56:05 +0100
Subject: [PATCH 3/5] move-ivcanon-pass-20170529.txt

---
 gcc/passes.def | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gcc/passes.def b/gcc/passes.def
index 10a18bf..beb350b 100644
--- a/gcc/passes.def
+++ b/gcc/passes.def
@@ -277,6 +277,7 @@ along with GCC; see the file COPYING3.  If not see
 	     empty loops.  Remove them now.  */
 	  NEXT_PASS (pass_cd_dce);
 	  NEXT_PASS (pass_record_bounds);
+	  NEXT_PASS (pass_iv_canon);
 	  NEXT_PASS (pass_loop_distribution);
 	  NEXT_PASS (pass_copy_prop);
 	  NEXT_PASS (pass_graphite);
@@ -286,7 +287,6 @@ along with GCC; see the file COPYING3.  If not see
 	      NEXT_PASS (pass_copy_prop);
 	      NEXT_PASS (pass_dce);
 	  POP_INSERT_PASSES ()
-	  NEXT_PASS (pass_iv_canon);
 	  NEXT_PASS (pass_parallelize_loops, false /* oacc_kernels_p */);
 	  NEXT_PASS (pass_expand_omp_ssa);
 	  NEXT_PASS (pass_ch_vect);
-- 
1.9.1

Re: [PATCH GCC][3/5]Move pass ivcanon upward in compilation process

[Richard Biener]

On Fri, Jun 2, 2017 at 1:51 PM, Bin Cheng <Bin.Cheng@arm.com> wrote:
> Hi,
> This patch moves pass ivcanon before loop distribution.  Pass loop split could create
> loops with limited niters.  Such loop should be unrolled before loop distribution (or graphite),
> rather than after.
>
> Bootstrap and test at O2/O3 on x86_64 and AArch64.  is it OK?

Ok.

Richard.

> Thanks,
> bin
> 2017-05-31  Bin Cheng  <bin.cheng@arm.com>
>
>         * passes.def (pass_iv_canon): Move before pass_loop_distribution.