<div dir="ltr">Hi Andrew,<div><br></div><div>Sorry for the delay. I had a read on the code around it and it seems that with your patch, the stack will always grow if there is a call and not a frame pointer. I'm not sure how this correlates with functions without stack setup, line __aeabi_read_tp, but it might bloat the stack for many other unrelated functions.</div>
<div><br></div><div style>However, I can see from the comments that adjusting the stack should be done on any function call, and I don't know that part of the code well enough, so if no one else has any objections, I'm ok with your patch.</div>
<div style><br></div><div style>cheers,</div><div style>--renato</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 18 April 2013 23:48, Andrew Turner <span dir="ltr"><<a href="mailto:andrew@fubar.geek.nz" target="_blank">andrew@fubar.geek.nz</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Thu, 18 Apr 2013 09:28:03 +0100<br>
Renato Golin <<a href="mailto:renato.golin@linaro.org">renato.golin@linaro.org</a>> wrote:<br>
<br>
> On 18 April 2013 03:35, Andrew Turner <<a href="mailto:andrew@fubar.geek.nz">andrew@fubar.geek.nz</a>> wrote:<br>
><br>
> > The attached patch makes tells the calling function it needs to<br>
> > adjust it's stack when it contains a call opcode.<br>
> ><br>
><br>
> Hi Andrew,<br>
><br>
> Isn't that a bit too generic a fix for such a specific case?<br>
<br>
</div></div>I don't think so. Clang should adjust the stack when a function calls<br>
another, this patch fixes the case where the function call doesn't have<br>
the frame setup or destroy opcodes around it. I'm not sure how common<br>
this is as I have only observed it with __aeabi_read_tp, and there may<br>
be other cases that have yet to been observed.<br>
<span class="HOEnZb"><font color="#888888"><br>
Andrew<br>
</font></span></blockquote></div><br></div>