Re: [PATCH 0/7] Support vector load/store with length

2020-05-27 Thread Segher Boessenkool
On Wed, May 27, 2020 at 09:25:43AM +0200, Richard Biener wrote: > On Tue, 26 May 2020, Segher Boessenkool wrote: > > On Tue, May 26, 2020 at 01:29:30PM +0100, Richard Sandiford wrote: > > > FWIW, I agree adding .LEN_LOAD and .LEN_STORE seems like a good > > > approach. I think it'll be more mainta

Re: [PATCH 0/7] Support vector load/store with length

2020-05-27 Thread Kewen.Lin via Gcc-patches
on 2020/5/27 下午3:25, Richard Biener wrote: > On Tue, 26 May 2020, Segher Boessenkool wrote: > >> Hi! >> >> On Tue, May 26, 2020 at 01:29:30PM +0100, Richard Sandiford wrote: >>> FWIW, I agree adding .LEN_LOAD and .LEN_STORE seems like a good >>> approach. I think it'll be more maintainable in the

Re: [PATCH 0/7] Support vector load/store with length

2020-05-27 Thread Richard Sandiford
Richard Biener writes: > On Tue, 26 May 2020, Jim Wilson wrote: > >> On Tue, May 26, 2020 at 12:12 AM Richard Biener wrote: >> > From a look at the series description below you seem to add a new way >> > of doing loads for this. Did you review other ISAs (those I'm not >> > familiar with myself

Re: [PATCH 0/7] Support vector load/store with length

2020-05-27 Thread Richard Biener
On Tue, 26 May 2020, Segher Boessenkool wrote: > Hi! > > On Tue, May 26, 2020 at 01:29:30PM +0100, Richard Sandiford wrote: > > FWIW, I agree adding .LEN_LOAD and .LEN_STORE seems like a good > > approach. I think it'll be more maintainable in the long run than > > trying to have .MASK_LOADs and

Re: [PATCH 0/7] Support vector load/store with length

2020-05-27 Thread Richard Biener
On Tue, 26 May 2020, Jim Wilson wrote: > On Tue, May 26, 2020 at 12:12 AM Richard Biener wrote: > > From a look at the series description below you seem to add a new way > > of doing loads for this. Did you review other ISAs (those I'm not > > familiar with myself too much are SVE, RISC-V and GC

Re: [PATCH 0/7] Support vector load/store with length

2020-05-26 Thread Segher Boessenkool
Hi! On Tue, May 26, 2020 at 01:29:30PM +0100, Richard Sandiford wrote: > FWIW, I agree adding .LEN_LOAD and .LEN_STORE seems like a good > approach. I think it'll be more maintainable in the long run than > trying to have .MASK_LOADs and .MASK_STOREs that need a special mask > operand. (That wou

Re: [PATCH 0/7] Support vector load/store with length

2020-05-26 Thread Jim Wilson
On Tue, May 26, 2020 at 12:12 AM Richard Biener wrote: > From a look at the series description below you seem to add a new way > of doing loads for this. Did you review other ISAs (those I'm not > familiar with myself too much are SVE, RISC-V and GCN) in GCC whether > they have similar support an

Re: [PATCH 0/7] Support vector load/store with length

2020-05-26 Thread Richard Sandiford
"Kewen.Lin" writes: > on 2020/5/26 下午5:44, Richard Biener wrote: >> On Tue, 26 May 2020, Kewen.Lin wrote: >> >>> Hi Richi, >>> >>> on 2020/5/26 下午3:12, Richard Biener wrote: On Tue, 26 May 2020, Kewen.Lin wrote: > Hi all, > > This patch set adds support for vector load/store

Re: [PATCH 0/7] Support vector load/store with length

2020-05-26 Thread Kewen.Lin via Gcc-patches
on 2020/5/26 下午5:44, Richard Biener wrote: > On Tue, 26 May 2020, Kewen.Lin wrote: > >> Hi Richi, >> >> on 2020/5/26 下午3:12, Richard Biener wrote: >>> On Tue, 26 May 2020, Kewen.Lin wrote: >>> Hi all, This patch set adds support for vector load/store with length, Power ISA 3.0

Re: [PATCH 0/7] Support vector load/store with length

2020-05-26 Thread Richard Biener
On Tue, 26 May 2020, Kewen.Lin wrote: > Hi Richi, > > on 2020/5/26 下午3:12, Richard Biener wrote: > > On Tue, 26 May 2020, Kewen.Lin wrote: > > > >> Hi all, > >> > >> This patch set adds support for vector load/store with length, Power > >> ISA 3.0 brings instructions lxvl/stxvl to perform vecto

Re: [PATCH 0/7] Support vector load/store with length

2020-05-26 Thread Kewen.Lin via Gcc-patches
Hi Richi, on 2020/5/26 下午3:12, Richard Biener wrote: > On Tue, 26 May 2020, Kewen.Lin wrote: > >> Hi all, >> >> This patch set adds support for vector load/store with length, Power >> ISA 3.0 brings instructions lxvl/stxvl to perform vector load/store with >> length, it's good to be exploited fo

Re: [PATCH 0/7] Support vector load/store with length

2020-05-26 Thread Richard Biener
On Tue, 26 May 2020, Kewen.Lin wrote: > Hi all, > > This patch set adds support for vector load/store with length, Power > ISA 3.0 brings instructions lxvl/stxvl to perform vector load/store with > length, it's good to be exploited for those cases we don't have enough > stuffs to fill in the who