Re: [x86 PATCH] PR target/31985: Improve memory operand use with doubleword add.

2023-06-16 Thread Uros Bizjak via Gcc-patches
On Fri, Jun 16, 2023 at 12:04 AM Roger Sayle wrote: > > > Hi Uros, > > > On the 7th June 2023, Uros Bizkak wrote: > > The register allocator considers the instruction-to-be-split as one > > instruction, so it > > can allocate output register to match an input register (or a register that > >

RE: [x86 PATCH] PR target/31985: Improve memory operand use with doubleword add.

2023-06-15 Thread Roger Sayle
Hi Uros, > On the 7th June 2023, Uros Bizkak wrote: > The register allocator considers the instruction-to-be-split as one > instruction, so it > can allocate output register to match an input register (or a register that > forms an > input address), So, you have to either add an early clobber

Re: [x86 PATCH] PR target/31985: Improve memory operand use with doubleword add.

2023-06-07 Thread Uros Bizjak via Gcc-patches
On Wed, Jun 7, 2023 at 8:32 AM Uros Bizjak wrote: > > On Wed, Jun 7, 2023 at 1:05 AM Roger Sayle wrote: > > > > > > This patch addresses the last remaining issue with PR target/31985, that > > GCC could make better use of memory addressing modes when implementing > > double word addition. This

Re: [x86 PATCH] PR target/31985: Improve memory operand use with doubleword add.

2023-06-07 Thread Uros Bizjak via Gcc-patches
On Wed, Jun 7, 2023 at 1:05 AM Roger Sayle wrote: > > > This patch addresses the last remaining issue with PR target/31985, that > GCC could make better use of memory addressing modes when implementing > double word addition. This is achieved by adding a define_insn_and_split > that combines an

[x86 PATCH] PR target/31985: Improve memory operand use with doubleword add.

2023-06-06 Thread Roger Sayle
This patch addresses the last remaining issue with PR target/31985, that GCC could make better use of memory addressing modes when implementing double word addition. This is achieved by adding a define_insn_and_split that combines an *add3_doubleword with a *concat3, so that the components of