Re: [PATCH v4] rs6000: Add load density heuristic

2021-09-09 Thread Kewen.Lin via Gcc-patches
on 2021/9/10 上午11:22, Kewen.Lin via Gcc-patches wrote: > Hi Segher and Bill, > > Thanks a lot for your reviews and helps! > > on 2021/9/10 上午1:19, Bill Schmidt wrote: >> On 9/9/21 11:11 AM, Segher Boessenkool wrote: >>> Hi! >>> >>> On Wed, Sep 08, 2021 at 02:57:14PM +0800, Kewen.Lin wrote: >>

Re: [PATCH v4] rs6000: Add load density heuristic

2021-09-09 Thread Kewen.Lin via Gcc-patches
Hi Segher and Bill, Thanks a lot for your reviews and helps! on 2021/9/10 上午1:19, Bill Schmidt wrote: > On 9/9/21 11:11 AM, Segher Boessenkool wrote: >> Hi! >> >> On Wed, Sep 08, 2021 at 02:57:14PM +0800, Kewen.Lin wrote: > +  /* If we have strided or elementwise loads into a vector, it's

Re: [PATCH v4] rs6000: Add load density heuristic

2021-09-09 Thread Segher Boessenkool
Hi! On Thu, Sep 09, 2021 at 12:19:28PM -0500, Bill Schmidt wrote: > On 9/9/21 11:11 AM, Segher Boessenkool wrote: > >On Wed, Sep 08, 2021 at 02:57:14PM +0800, Kewen.Lin wrote: > >>+ /* If we have strided or elementwise loads into a vector, it's > >"strided" is not a word: it properly is "stri

Re: [PATCH v4] rs6000: Add load density heuristic

2021-09-09 Thread Bill Schmidt via Gcc-patches
On 9/9/21 12:19 PM, Bill Schmidt wrote: On 9/9/21 11:11 AM, Segher Boessenkool wrote: Hi! On Wed, Sep 08, 2021 at 02:57:14PM +0800, Kewen.Lin wrote: + /* If we have strided or elementwise loads into a vector, it's +possible to be bounded by latency and execution resources for +

Re: [PATCH v4] rs6000: Add load density heuristic

2021-09-09 Thread Bill Schmidt via Gcc-patches
On 9/9/21 11:11 AM, Segher Boessenkool wrote: Hi! On Wed, Sep 08, 2021 at 02:57:14PM +0800, Kewen.Lin wrote: + /* If we have strided or elementwise loads into a vector, it's +possible to be bounded by latency and execution resources for +many scalar loads. Try to account f

Re: [PATCH v4] rs6000: Add load density heuristic

2021-09-09 Thread Segher Boessenkool
Hi! On Wed, Sep 08, 2021 at 02:57:14PM +0800, Kewen.Lin wrote: > >>+ /* If we have strided or elementwise loads into a vector, it's > >>+possible to be bounded by latency and execution resources for > >>+many scalar loads. Try to account for this by scaling the > >>+construction

Re: [PATCH v4] rs6000: Add load density heuristic

2021-09-08 Thread Kewen.Lin via Gcc-patches
on 2021/9/8 下午2:57, Kewen.Lin via Gcc-patches wrote: > Hi Bill, > > Thanks for the review comments! > > on 2021/9/3 下午11:57, Bill Schmidt wrote: >> Hi Kewen, >> >> Sorry that we lost track of this patch!  The heuristic approach looks good.  >> It is limited in scope and won't kick in often, and