<div dir="ltr">This change is in the code that is not called normally, in contrast to PEI::calculateCallFrameInfo, so there is no target for which it could fix a bug. On the other hand there are stackless architectures supported in llvm that could be affected.<div><br></div><div>If this is not sufficient for justification, I will revert this change, as I don't know particular problem.</div><div>Sorry for troubles.<br><div><div><div class="gmail_extra"><br clear="all"><div><div class="m_-8045683112394495787gmail_signature">Thanks,<br>--Serge<br></div></div>
<br><div class="gmail_quote">2017-04-20 21:19 GMT+07:00 Davide Italiano <span dir="ltr"><<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="m_-8045683112394495787gmail-">On Wed, Apr 19, 2017 at 10:35 PM, Serge Pavlov <<a href="mailto:sepavloff@gmail.com" target="_blank">sepavloff@gmail.com</a>> wrote:<br>
> These lines were copied from PEI::calculateCallFrameInfo. To test them we<br>
> need a target that do not use frame instructions, it could be a stackless<br>
> architecture. I cannot find such in intree targets.<br>
> Anyway the change is obvious and useful for some targets.<br>
><br>
<br>
</span>I can see this patch committed if somebody is actually leveraging it<br>
(even out-of-tree). Do you have such an example?<br>
Otherwise, it's pretty much code thrown in the tree (which is not ideal).<br>
<br>
--<br>
Davide<br>
</blockquote></div><br></div></div></div></div></div>