Re: [PATCH] vect/test: Don't check for epilogue loop [PR97075]

2020-09-21 Thread Kewen.Lin via Gcc-patches
Hi Richard, on 2020/9/21 下午2:50, Richard Sandiford wrote: > "Kewen.Lin" writes: >> Hi Richard, >>> "Kewen.Lin" writes: Hi, The commit r11-3230 brings a nice improvement to use full vectors instead of partial vectors when available. But it caused some vector with length

Re: [PATCH] vect/test: Don't check for epilogue loop [PR97075]

2020-09-21 Thread Richard Sandiford
Andrea Corallo writes: > Richard Sandiford writes: > [...] >> Andrea, how should we handle this? Is it something you'd have time to >> look at? > > Hi Richard, > > I've not OK, NP. In that case I'll give it a go. > but FWIW your observations here and on today's mail make alot > of sense to me

Re: [PATCH] vect/test: Don't check for epilogue loop [PR97075]

2020-09-21 Thread Andrea Corallo
Richard Sandiford writes: [...] > Andrea, how should we handle this? Is it something you'd have time to > look at? Hi Richard, I've not but FWIW your observations here and on today's mail make alot of sense to me. We maybe want to install Kewen's fix anyway while we rework this logic? Andre

Re: [PATCH] vect/test: Don't check for epilogue loop [PR97075]

2020-09-20 Thread Richard Sandiford
"Kewen.Lin" writes: > Hi Richard, >> "Kewen.Lin" writes: >>> Hi, >>> >>> The commit r11-3230 brings a nice improvement to use full >>> vectors instead of partial vectors when available. But >>> it caused some vector with length test cases to fail on >>> Power. >>> >>> The failure on gcc.target/p

Re: [PATCH] vect/test: Don't check for epilogue loop [PR97075]

2020-09-20 Thread Kewen.Lin via Gcc-patches
Hi Richard, > "Kewen.Lin" writes: >> Hi, >> >> The commit r11-3230 brings a nice improvement to use full >> vectors instead of partial vectors when available. But >> it caused some vector with length test cases to fail on >> Power. >> >> The failure on gcc.target/powerpc/p9-vec-length-epil-7.c >

Re: [PATCH] vect/test: Don't check for epilogue loop [PR97075]

2020-09-18 Thread Richard Sandiford
Thanks for looking at this. "Kewen.Lin" writes: > Hi, > > The commit r11-3230 brings a nice improvement to use full > vectors instead of partial vectors when available. But > it caused some vector with length test cases to fail on > Power. > > The failure on gcc.target/powerpc/p9-vec-length-epil

Re: [PATCH] vect/test: Don't check for epilogue loop [PR97075]

2020-09-18 Thread Segher Boessenkool
On Fri, Sep 18, 2020 at 10:37:47AM +0800, Kewen.Lin wrote: > The commit r11-3230 brings a nice improvement to use full > vectors instead of partial vectors when available. But > it caused some vector with length test cases to fail on > Power. > > The failure on gcc.target/powerpc/p9-vec-length-ep

[PATCH] vect/test: Don't check for epilogue loop [PR97075]

2020-09-17 Thread Kewen.Lin via Gcc-patches
Hi, The commit r11-3230 brings a nice improvement to use full vectors instead of partial vectors when available. But it caused some vector with length test cases to fail on Power. The failure on gcc.target/powerpc/p9-vec-length-epil-7.c exposed one issue that: we call function vect_need_peeling