On 8/17/23 07:19, senthilkumar.selva...@microchip.com wrote:
On Wed, 2023-08-16 at 12:13 -0400, Vladimir Makarov wrote:
EXTERNAL EMAIL: Do not click links or open attachments unless you know the
content is safe
The attached patch fixes recently found wrong insn removal in LRA port
for AVR.
The patch was successfully tested and bootstrapped on x86-64 and aarch64.
Hi Vladimir,
Thanks for working on this. After applying the patch, I'm seeing that the
pseudo in the frame pointer that got spilled is taking up the same stack
slot that was already assigned to a spilled pseudo, and that is causing
execution
failure (it is also causing a crash when building libgcc for avr)
...
I tried a hacky workaround (see patch below) to create a new stack slot and
assign the spilled pseudo to it, and that works.
Not sure if that's the right way to do it though.
The general way of solution is right but I've just committed a different
version of the patch.
