Re: [PATCH] Do not count unused scalar use when marking STMT_VINFO_LIVE_P [PR113091]

2024-01-12 Thread Richard Biener
; +} > } > > /* Determine whether we can vectorize the reduction epilogue for INSTANCE. > */ > @@ -6684,17 +6823,7 @@ vect_slp_analyze_operations (vec_info *vinfo) > >/* Compute vectorizable live stmts. */ >if (bb_vec_info bb_vinfo = dyn_cast (vinfo)) >

Re: [PATCH] Do not count unused scalar use when marking STMT_VINFO_LIVE_P [PR113091]

2024-01-11 Thread Feng Xue OS
11, 2024 5:52 PM To: Feng Xue OS; Richard Sandiford Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH] Do not count unused scalar use when marking STMT_VINFO_LIVE_P [PR113091] On Thu, Jan 11, 2024 at 10:46 AM Richard Biener wrote: > > On Fri, Dec 29, 2023 at 11:29 AM Feng Xue OS > wrote: &

Re: [PATCH] Do not count unused scalar use when marking STMT_VINFO_LIVE_P [PR113091]

2024-01-11 Thread Richard Biener
On Thu, Jan 11, 2024 at 10:46 AM Richard Biener wrote: > > On Fri, Dec 29, 2023 at 11:29 AM Feng Xue OS > wrote: > > > > This patch is meant to fix over-estimation about SLP vector-to-scalar cost > > for > > STMT_VINFO_LIVE_P statement. When pattern recognition is involved, a > > statement

Re: [PATCH] Do not count unused scalar use when marking STMT_VINFO_LIVE_P [PR113091]

2024-01-11 Thread Richard Biener
On Fri, Dec 29, 2023 at 11:29 AM Feng Xue OS wrote: > > This patch is meant to fix over-estimation about SLP vector-to-scalar cost for > STMT_VINFO_LIVE_P statement. When pattern recognition is involved, a > statement whose definition is consumed in some pattern, may not be > included in the

PING: [PATCH] Do not count unused scalar use when marking STMT_VINFO_LIVE_P [PR113091]

2024-01-10 Thread Feng Xue OS
Hi, Richard, Would you please talk a look at this patch? Thanks, Feng From: Feng Xue OS Sent: Friday, December 29, 2023 6:28 PM To: gcc-patches@gcc.gnu.org Subject: [PATCH] Do not count unused scalar use when marking STMT_VINFO_LIVE_P [PR113091

[PATCH] Do not count unused scalar use when marking STMT_VINFO_LIVE_P [PR113091]

2023-12-29 Thread Feng Xue OS
This patch is meant to fix over-estimation about SLP vector-to-scalar cost for STMT_VINFO_LIVE_P statement. When pattern recognition is involved, a statement whose definition is consumed in some pattern, may not be included in the final replacement pattern statements, and would be skipped when