On 14-01-01 09:23 AM, Jan Hubicka wrote:
Here insn 765 gets adjusted for insn766 wihtout updating REG_ARGS_SIZE.
This is IMO not bug in scheduler but bug in fact that insn 775 should not have 
the note on it
when it is not adjusting stack pointer. This is because originally it was push 
instruction,
but combine-stack-adjustments turned it into move.
I think it should remove the ARG_SIZE note when doing so, since argument block 
size should
be corrently annotated by preceeding stack adjustment insn.
Bootstrapped/regtested x86_64-linux. Did not double check it is solves the ARM 
issue
reported in PR, too.
FWIW, I tested your patch and it doesn't address the ICE on ARM.

Regards,

Ryan Mansfield

Reply via email to