Re: [PATCH] RISC-V: costs: handle BSWAP

2022-11-09 Thread Philipp Tomsich
Applied to master, with the commit-message regenerated as: gcc/ChangeLog: * config/riscv/riscv.cc (riscv_rtx_costs): Add BSWAP. Thanks, Philipp. On Wed, 9 Nov 2022 at 04:15, Palmer Dabbelt wrote: > > On Tue, 08 Nov 2022 18:57:26 PST (-0800), jeffreya...@gmail.com wrote: > > > >

Re: [PATCH] RISC-V: costs: handle BSWAP

2022-11-09 Thread Philipp Tomsich
On Wed, 9 Nov 2022 at 05:43, Andrew Pinski wrote: > > On Tue, Nov 8, 2022 at 7:16 PM Palmer Dabbelt wrote: > > > > On Tue, 08 Nov 2022 18:57:26 PST (-0800), jeffreya...@gmail.com wrote: > > > > > > On 11/8/22 12:54, Philipp Tomsich wrote: > > >> The BSWAP operation is not handled in rtx_costs. Ad

Re: [PATCH] RISC-V: costs: handle BSWAP

2022-11-08 Thread Palmer Dabbelt
On Tue, 08 Nov 2022 20:43:20 PST (-0800), pins...@gmail.com wrote: On Tue, Nov 8, 2022 at 7:16 PM Palmer Dabbelt wrote: On Tue, 08 Nov 2022 18:57:26 PST (-0800), jeffreya...@gmail.com wrote: > > On 11/8/22 12:54, Philipp Tomsich wrote: >> The BSWAP operation is not handled in rtx_costs. Add it

Re: [PATCH] RISC-V: costs: handle BSWAP

2022-11-08 Thread Andrew Pinski via Gcc-patches
On Tue, Nov 8, 2022 at 7:16 PM Palmer Dabbelt wrote: > > On Tue, 08 Nov 2022 18:57:26 PST (-0800), jeffreya...@gmail.com wrote: > > > > On 11/8/22 12:54, Philipp Tomsich wrote: > >> The BSWAP operation is not handled in rtx_costs. Add it. > >> > >> With Zbb, BSWAP for XLEN is a single instruction;

Re: [PATCH] RISC-V: costs: handle BSWAP

2022-11-08 Thread Palmer Dabbelt
On Tue, 08 Nov 2022 18:57:26 PST (-0800), jeffreya...@gmail.com wrote: On 11/8/22 12:54, Philipp Tomsich wrote: The BSWAP operation is not handled in rtx_costs. Add it. With Zbb, BSWAP for XLEN is a single instruction; for smaller modes, it will expand into two. gcc/ChangeLog: * con

Re: [PATCH] RISC-V: costs: handle BSWAP

2022-11-08 Thread Jeff Law via Gcc-patches
On 11/8/22 12:54, Philipp Tomsich wrote: The BSWAP operation is not handled in rtx_costs. Add it. With Zbb, BSWAP for XLEN is a single instruction; for smaller modes, it will expand into two. gcc/ChangeLog: * config/riscv/riscv.c (rtx_costs): Add BSWAP. OK. Jeff

[PATCH] RISC-V: costs: handle BSWAP

2022-11-08 Thread Philipp Tomsich
The BSWAP operation is not handled in rtx_costs. Add it. With Zbb, BSWAP for XLEN is a single instruction; for smaller modes, it will expand into two. gcc/ChangeLog: * config/riscv/riscv.c (rtx_costs): Add BSWAP. --- gcc/config/riscv/riscv.cc | 10 ++ 1 file changed, 10 insert