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: [PATCH GCC 5/9]Put copied loop after its preheader and after the original loop's latch in basic block link list

From: Jeff Law <law at redhat dot com>
To: Bin Cheng <Bin dot Cheng at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Cc: nd <nd at arm dot com>
Date: Wed, 7 Sep 2016 06:28:21 -0600
Subject: Re: [PATCH GCC 5/9]Put copied loop after its preheader and after the original loop's latch in basic block link list
Authentication-results: sourceware.org; auth=none
References: <AM4PR0802MB2163460418952FB1C4A833ABE7F90@AM4PR0802MB2163.eurprd08.prod.outlook.com>

On 09/06/2016 12:52 PM, Bin Cheng wrote:

Hi,
This simple patch changes slpeel_tree_duplicate_loop_edge_cfg by putting copied loop after its new preheader and after the original loop's latch in basic block's linked list.  It doesn't change CFG at all, but makes the dump cfg a little bit easier to read.  I assume this is good for basic block reordering too?

Thanks,
bin

2016-09-01  Bin Cheng  <bin.cheng@arm.com>

	* tree-vect-loop-manip.c (slpeel_tree_duplicate_loop_to_edge_cfg): Put
	duplicated loop after its preheader and after the original loop.

In theory bb reordering ought to clean this up. But better to generategood code from the start when it's easy to do so.


OK.

jeff

References:
- [PATCH GCC 5/9]Put copied loop after its preheader and after the original loop's latch in basic block link list
  - From: Bin Cheng

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