Re: [PATCH 5/7] vect: Support vector load/store with length in vectorizer

2020-05-29 Thread Segher Boessenkool
Hi! On Fri, May 29, 2020 at 09:32:49AM +0100, Richard Sandiford wrote: > There's nothing to stop us using masks and lengths in the same loop > in future if we need to. It would “just” be a case of setting up both > the masks and the lengths in vect_set_loop_condition. But the point is > that

Re: [PATCH 5/7] vect: Support vector load/store with length in vectorizer

2020-05-29 Thread Richard Sandiford
"Kewen.Lin" writes: > on 2020/5/27 下午6:02, Richard Sandiford wrote: >> "Kewen.Lin" writes: >>> Hi Richard, >>> >>> Thanks for your comments! >>> >>> on 2020/5/26 锟斤拷锟斤拷8:49, Richard Sandiford wrote: "Kewen.Lin" writes: > @@ -626,6 +645,12 @@ public: >/* True if have decided to

Re: [PATCH 5/7] vect: Support vector load/store with length in vectorizer

2020-05-27 Thread Kewen.Lin via Gcc-patches
on 2020/5/27 下午6:02, Richard Sandiford wrote: > "Kewen.Lin" writes: >> Hi Richard, >> >> Thanks for your comments! >> >> on 2020/5/26 锟斤拷锟斤拷8:49, Richard Sandiford wrote: >>> "Kewen.Lin" writes: @@ -626,6 +645,12 @@ public: /* True if have decided to use a fully-masked loop. */

Re: [PATCH 5/7] vect: Support vector load/store with length in vectorizer

2020-05-27 Thread Richard Sandiford
"Kewen.Lin" writes: > Hi Richard, > > Thanks for your comments! > > on 2020/5/26 锟斤拷锟斤拷8:49, Richard Sandiford wrote: >> "Kewen.Lin" writes: >>> @@ -626,6 +645,12 @@ public: >>>/* True if have decided to use a fully-masked loop. */ >>>bool fully_masked_p; >>> >>> + /* Records whether

Re: [PATCH 5/7] vect: Support vector load/store with length in vectorizer

2020-05-27 Thread Kewen.Lin via Gcc-patches
Hi Richard, Thanks for your comments! on 2020/5/26 下午8:49, Richard Sandiford wrote: > "Kewen.Lin" writes: >> @@ -626,6 +645,12 @@ public: >>/* True if have decided to use a fully-masked loop. */ >>bool fully_masked_p; >> >> + /* Records whether we still have the option of using a

Re: [PATCH 5/7] vect: Support vector load/store with length in vectorizer

2020-05-26 Thread Richard Sandiford
Richard Sandiford writes: > "Kewen.Lin" writes: >> @@ -626,6 +645,12 @@ public: >>/* True if have decided to use a fully-masked loop. */ >>bool fully_masked_p; >> >> + /* Records whether we still have the option of using a length access >> loop. */ >> + bool can_with_length_p; >>

Re: [PATCH 5/7] vect: Support vector load/store with length in vectorizer

2020-05-26 Thread Richard Sandiford
"Kewen.Lin" writes: > @@ -626,6 +645,12 @@ public: >/* True if have decided to use a fully-masked loop. */ >bool fully_masked_p; > > + /* Records whether we still have the option of using a length access loop. > */ > + bool can_with_length_p; > + > + /* True if have decided to use

[PATCH 5/7] vect: Support vector load/store with length in vectorizer

2020-05-25 Thread Kewen.Lin via Gcc-patches
gcc/ChangeLog 2020-MM-DD Kewen Lin * doc/invoke.texi (vect-with-length-scope): Document new option. * params.opt (vect-with-length-scope): New. * tree-vect-loop-manip.c (vect_set_loop_lens_directly): New function. (vect_set_loop_condition_len): Likewise.