Re: [PATCH v4] vect/rs6000: Support vector with length cost modeling

2020-07-31 Thread Kewen.Lin via Gcc-patches
on 2020/7/31 下午9:01, Richard Biener wrote: > On Fri, Jul 31, 2020 at 2:37 PM Kewen.Lin wrote: >> >> Hi Richards, >> >> on 2020/7/31 下午7:20, Richard Biener wrote: >>> On Fri, Jul 31, 2020 at 1:03 PM Richard Sandiford >>> wrote: "Kewen.Lin" writes: >>> + bool niters_known_p = LO

Re: [PATCH v4] vect/rs6000: Support vector with length cost modeling

2020-07-31 Thread Richard Biener via Gcc-patches
On Fri, Jul 31, 2020 at 2:37 PM Kewen.Lin wrote: > > Hi Richards, > > on 2020/7/31 下午7:20, Richard Biener wrote: > > On Fri, Jul 31, 2020 at 1:03 PM Richard Sandiford > > wrote: > >> > >> "Kewen.Lin" writes: > > + bool niters_known_p = LOOP_VINFO_NITERS_KNOWN_P (loop_vinfo); > > +

Re: [PATCH v4] vect/rs6000: Support vector with length cost modeling

2020-07-31 Thread Kewen.Lin via Gcc-patches
Hi Richards, on 2020/7/31 下午7:20, Richard Biener wrote: > On Fri, Jul 31, 2020 at 1:03 PM Richard Sandiford > wrote: >> >> "Kewen.Lin" writes: > + bool niters_known_p = LOOP_VINFO_NITERS_KNOWN_P (loop_vinfo); > + bool need_iterate_p > + = (!LOOP_VINFO_EPILOGUE_P (loop_vin

Re: [PATCH v4] vect/rs6000: Support vector with length cost modeling

2020-07-31 Thread Richard Biener via Gcc-patches
On Fri, Jul 31, 2020 at 1:03 PM Richard Sandiford wrote: > > "Kewen.Lin" writes: > >>> + bool niters_known_p = LOOP_VINFO_NITERS_KNOWN_P (loop_vinfo); > >>> + bool need_iterate_p > >>> + = (!LOOP_VINFO_EPILOGUE_P (loop_vinfo) > >>> + && !vect_known_niters_smaller_than_vf (loop_vi

Re: [PATCH v4] vect/rs6000: Support vector with length cost modeling

2020-07-31 Thread Richard Sandiford
"Kewen.Lin" writes: >>> + bool niters_known_p = LOOP_VINFO_NITERS_KNOWN_P (loop_vinfo); >>> + bool need_iterate_p >>> + = (!LOOP_VINFO_EPILOGUE_P (loop_vinfo) >>> + && !vect_known_niters_smaller_than_vf (loop_vinfo)); >>> + >>> + /* Init min/max, shift and minus cost relative

Re: [PATCH v4] vect/rs6000: Support vector with length cost modeling

2020-07-28 Thread Kewen.Lin via Gcc-patches
Hi Richard, Thanks for the comments! on 2020/7/27 下午9:40, Richard Sandiford wrote: > "Kewen.Lin" writes: [snip] >> + bool niters_known_p = LOOP_VINFO_NITERS_KNOWN_P (loop_vinfo); >> + bool need_iterate_p >> += (!LOOP_VINFO_EPILOGUE_P (loop_vinfo) >> + && !vect_known_niters_

Re: [PATCH v4] vect/rs6000: Support vector with length cost modeling

2020-07-27 Thread Richard Sandiford
"Kewen.Lin" writes: > Hi Richard, > > Thanks for the review again! > > on 2020/7/25 上午12:21, Richard Sandiford wrote: >> "Kewen.Lin" writes: >> >> Thanks, the rearrangement of the existing code looks good. Could you >> split that and the new LOOP_VINFO_FULLY_WITH_LENGTH_P (loop_vinfo) stuff >>