Re: [PATCH v6 3/3] PR80791 Consider doloop cmp use in ivopts

on 2019/9/12 下午4:14, Richard Biener wrote: > On Wed, 11 Sep 2019, Kewen.Lin wrote: > >> Hi, >> >> Sorry for the late update. I've updated the words of target hooks part. >> >> Could someone help to review it? Thanks in advance! >> >> By the way, as previous emails in this thread, Bin has approve

Re: [PATCH v6 3/3] PR80791 Consider doloop cmp use in ivopts

On Wed, 11 Sep 2019, Kewen.Lin wrote: > Hi, > > Sorry for the late update. I've updated the words of target hooks part. > > Could someone help to review it? Thanks in advance! > > By the way, as previous emails in this thread, Bin has approved the IVOPTs > part, while Segher has approved the

Re: [PATCH v6 3/3] PR80791 Consider doloop cmp use in ivopts

Hi, Sorry for the late update. I've updated the words of target hooks part. Could someone help to review it? Thanks in advance! By the way, as previous emails in this thread, Bin has approved the IVOPTs part, while Segher has approved the rs6000 part. Thanks, Kewen - gcc/ChangeLog 201

Re: [PATCH v6 3/3] PR80791 Consider doloop cmp use in ivopts

Hi Bin, on 2019/8/23 下午5:43, Bin.Cheng wrote: > On Fri, Aug 23, 2019 at 4:27 PM Kewen.Lin wrote: >> >> Hi Bin >> >> on 2019/8/23 上午10:19, Bin.Cheng wrote: >>> On Thu, Aug 22, 2019 at 3:09 PM Kewen.Lin wrote: Hi Bin, on 2019/8/22 下午1:46, Bin.Cheng wrote: > On Thu, Aug 22,

Re: [PATCH v6 3/3] PR80791 Consider doloop cmp use in ivopts

Hi! On Fri, Aug 23, 2019 at 05:43:32PM +0800, Bin.Cheng wrote: > On Fri, Aug 23, 2019 at 4:27 PM Kewen.Lin wrote: > Not sure if non-ivopts parts are already approved? If so, the patch > is okay with above issues addressed. The rs6000 part is fine. The target.def entries need some spell check a

Re: [PATCH v6 3/3] PR80791 Consider doloop cmp use in ivopts

On Fri, Aug 23, 2019 at 4:27 PM Kewen.Lin wrote: > > Hi Bin > > on 2019/8/23 上午10:19, Bin.Cheng wrote: > > On Thu, Aug 22, 2019 at 3:09 PM Kewen.Lin wrote: > >> > >> Hi Bin, > >> > >> on 2019/8/22 下午1:46, Bin.Cheng wrote: > >>> On Thu, Aug 22, 2019 at 11:18 AM Kewen.Lin wrote: > > Hi B

Re: [PATCH v6 3/3] PR80791 Consider doloop cmp use in ivopts

Hi Bin on 2019/8/23 上午10:19, Bin.Cheng wrote: > On Thu, Aug 22, 2019 at 3:09 PM Kewen.Lin wrote: >> >> Hi Bin, >> >> on 2019/8/22 下午1:46, Bin.Cheng wrote: >>> On Thu, Aug 22, 2019 at 11:18 AM Kewen.Lin wrote: Hi Bin, Thanks for your time! on 2019/8/21 下午8:32, Bin.Ch

Re: [PATCH v6 3/3] PR80791 Consider doloop cmp use in ivopts

On Thu, Aug 22, 2019 at 3:09 PM Kewen.Lin wrote: > > Hi Bin, > > on 2019/8/22 下午1:46, Bin.Cheng wrote: > > On Thu, Aug 22, 2019 at 11:18 AM Kewen.Lin wrote: > >> > >> Hi Bin, > >> > >> Thanks for your time! > >> > >> on 2019/8/21 下午8:32, Bin.Cheng wrote: > >>> On Wed, Aug 14, 2019 at 3:23 PM Kewe

Re: [PATCH v6 3/3] PR80791 Consider doloop cmp use in ivopts

Hi Bin, on 2019/8/22 下午1:46, Bin.Cheng wrote: > On Thu, Aug 22, 2019 at 11:18 AM Kewen.Lin wrote: >> >> Hi Bin, >> >> Thanks for your time! >> >> on 2019/8/21 下午8:32, Bin.Cheng wrote: >>> On Wed, Aug 14, 2019 at 3:23 PM Kewen.Lin wrote: Hi! Comparing to the previous versions

Re: [PATCH v6 3/3] PR80791 Consider doloop cmp use in ivopts

On Thu, Aug 22, 2019 at 11:18 AM Kewen.Lin wrote: > > Hi Bin, > > Thanks for your time! > > on 2019/8/21 下午8:32, Bin.Cheng wrote: > > On Wed, Aug 14, 2019 at 3:23 PM Kewen.Lin wrote: > >> > >> Hi! > >> > >> Comparing to the previous versions of implementation mainly based on the > >> existing IV

Re: [PATCH v6 3/3] PR80791 Consider doloop cmp use in ivopts

Hi Bin, Thanks for your time! on 2019/8/21 下午8:32, Bin.Cheng wrote: > On Wed, Aug 14, 2019 at 3:23 PM Kewen.Lin wrote: >> >> Hi! >> >> Comparing to the previous versions of implementation mainly based on the >> existing IV cands but zeroing the related group/use cost, this new one is >> based >

Re: [PATCH v6 3/3] PR80791 Consider doloop cmp use in ivopts

On Wed, Aug 14, 2019 at 3:23 PM Kewen.Lin wrote: > > Hi! > > Comparing to the previous versions of implementation mainly based on the > existing IV cands but zeroing the related group/use cost, this new one is > based > on Richard and Segher's suggestion introducing one doloop dedicated IV cand.

[PATCH v6 3/3] PR80791 Consider doloop cmp use in ivopts

Hi! Comparing to the previous versions of implementation mainly based on the existing IV cands but zeroing the related group/use cost, this new one is based on Richard and Segher's suggestion introducing one doloop dedicated IV cand. Some key points are listed below: 1) New field doloop_p in