Re: [PATCH 3/7] riscv: Enable overlap-by-pieces via tune param

2022-11-14 Thread Christoph Müllner
On Mon, Nov 14, 2022 at 8:04 PM Jeff Law wrote: > > On 11/14/22 01:29, Christoph Müllner wrote: > > > > On Mon, Nov 14, 2022 at 8:59 AM Philipp Tomsich > wrote: > >> On Mon, 14 Nov 2022 at 03:48, Vineet Gupta wrote: >> > >> > >> > >> > On 11/13/22 15:05, Christoph Muellner wrote: >> > > >> > >

Re: [PATCH 3/7] riscv: Enable overlap-by-pieces via tune param

2022-11-14 Thread Jeff Law via Gcc-patches
On 11/14/22 01:29, Christoph Müllner wrote: On Mon, Nov 14, 2022 at 8:59 AM Philipp Tomsich wrote: On Mon, 14 Nov 2022 at 03:48, Vineet Gupta wrote: > > > > On 11/13/22 15:05, Christoph Muellner wrote: > > > > +static bool > > +riscv_overlap_op_by_piec

Re: [PATCH 3/7] riscv: Enable overlap-by-pieces via tune param

2022-11-14 Thread Christoph Müllner
On Mon, Nov 14, 2022 at 8:59 AM Philipp Tomsich wrote: > On Mon, 14 Nov 2022 at 03:48, Vineet Gupta wrote: > > > > > > > > On 11/13/22 15:05, Christoph Muellner wrote: > > > > > > +static bool > > > +riscv_overlap_op_by_pieces (void) > > > +{ > > > + return tune_param->overlap_op_by_pieces; > >

Re: [PATCH 3/7] riscv: Enable overlap-by-pieces via tune param

2022-11-14 Thread Philipp Tomsich
On Mon, 14 Nov 2022 at 03:48, Vineet Gupta wrote: > > > > On 11/13/22 15:05, Christoph Muellner wrote: > > > > +static bool > > +riscv_overlap_op_by_pieces (void) > > +{ > > + return tune_param->overlap_op_by_pieces; > > Does this not need to be gated on unaligned access enabled as well. I assum

Re: [PATCH 3/7] riscv: Enable overlap-by-pieces via tune param

2022-11-13 Thread Vineet Gupta
On 11/13/22 15:05, Christoph Muellner wrote: +static bool +riscv_overlap_op_by_pieces (void) +{ + return tune_param->overlap_op_by_pieces; Does this not need to be gated on unaligned access enabled as well. -Vineet