> -----Original Message-----
> From: wangjue <[email protected]>
> Sent: Thursday, September 3, 2026 7:23 PM
> To: Cui, Lili <[email protected]>; Jeff Law <[email protected]>;
> gcc-patches <[email protected]>
> Cc: rdapp.gcc <[email protected]>; wangjue.wangjue
> <[email protected]>
> Subject: Re: [PATCH v4] ifcvt: Account for parallelism when costing noce
> sequences
> 
> Hi Lili
> 
> > I collected performance data with this patch on an x86 machine, with SPEC
> CPU2017/2026, there is no regression.
> Thanks for running the performance tests.
> 
> 
> >I have a slight concern: Why is only the new sequence costed in parallel, and
> not the original sequence? This leaves the comparison unfair.
> 
> That sounds like a reasonable concern. I will experiment locally with changing
> how the serial costs of the then and else blocks are calculated, and then 
> rerun
> the benchmarks to evaluate the performance impact.
> 
Sounds good, thanks!

BRs,
Lili

> 
> Best regards,
> Wang Jue
> 
> 

Reply via email to