Re: [PATCH, rs6000] Make load cost more in vectorization cost for P8/P9

2019-11-06 Thread Segher Boessenkool
Hi! On Tue, Nov 05, 2019 at 10:14:46AM +0800, Kewen.Lin wrote: > >> + benefits were observed on Power8 and up, we can unify it if similar > >> + profits are measured on Power6 and Power7. */ > >> + if (TARGET_P8_VECTOR) > >> +return 2; > >> + else > >> +return 1; > > > > Hrm, b

Re: [PATCH, rs6000] Make load cost more in vectorization cost for P8/P9

2019-11-04 Thread Kewen.Lin
Hi Segher, Thanks for the comments! on 2019/11/5 上午4:21, Segher Boessenkool wrote: > Hi! > > On Mon, Nov 04, 2019 at 03:16:06PM +0800, Kewen.Lin wrote: >> To align with rs6000_insn_cost costing more for load type insns, > > (Which itself has history in rs6000_rtx_costs). > >> this patch is to

Re: [PATCH, rs6000] Make load cost more in vectorization cost for P8/P9

2019-11-04 Thread Segher Boessenkool
Hi! On Mon, Nov 04, 2019 at 03:16:06PM +0800, Kewen.Lin wrote: > To align with rs6000_insn_cost costing more for load type insns, (Which itself has history in rs6000_rtx_costs). > this patch is to make load insns cost more in vectorization cost > function. Considering that the result of load us

[PATCH, rs6000] Make load cost more in vectorization cost for P8/P9

2019-11-03 Thread Kewen.Lin
Hi, To align with rs6000_insn_cost costing more for load type insns, this patch is to make load insns cost more in vectorization cost function. Considering that the result of load usually is used somehow later (true-dep) but store won't, we keep the store as before. The SPEC2017 performance eval