Hi, thanks for the feedback!
It turned out to be an endianness issue and we needed to treat the
call to `store_bit_field` differently for machines with
BITS_BIG_ENDIAN != BYTES_BIG_ENDIAN, like H8.
I've sent a new version
(https://gcc.gnu.org/pipermail/gcc-patches/2024-November/668214.html).
Than
On 10/23/24 8:27 AM, Konstantinos Eleftheriou wrote:
From: kelefth
This pass detects cases of expensive store forwarding and tries to avoid them
by reordering the stores and using suitable bit insertion sequences.
For example it can transform this:
strbw2, [x1, 1]
ldr x0
On Wed, Oct 23, 2024 at 04:27:29PM +0200, Konstantinos Eleftheriou wrote:
Just random ChangeLog formatting nits, not actual patch review:
> gcc/ChangeLog:
>
> * Makefile.in: Add avoid-store-forwarding.o
Missing . at the end. Though, you should really also mention
what you're changing, so
From: kelefth
This pass detects cases of expensive store forwarding and tries to avoid them
by reordering the stores and using suitable bit insertion sequences.
For example it can transform this:
strbw2, [x1, 1]
ldr x0, [x1] # Expensive store forwarding to larger load.
To
