<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sun, Jul 31, 2016 at 10:32 PM, Gerolf Hoflehner <span dir="ltr"><<a href="mailto:ghoflehner@apple.com" target="_blank">ghoflehner@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><span class="gmail-"><br>
> On Jul 31, 2016, at 1:46 AM, Amjad Aboud <<a href="mailto:amjad.aboud@intel.com">amjad.aboud@intel.com</a>> wrote:<br>
><br>
> aaboud added a comment.<br>
><br>
>> ISTM that the DWARF spec intended such thunks to be encoded as `DW_AT_trampoline`.  That seems more appropriate than relying on codegen emitting a tailcall.  This way the debugger can make the policy decision of whether or not thunks should show up in the backtrace.<br>
><br>
>><br>
><br>
>> In any case, correctness must always trump all else.  Reverting to green should take precedence over a QoI bug like PR24235.<br>
><br>
><br>
> I agree to the revert, though I am not sure about the new test, it looks too complected, especially the command line.<br>
<br>
</span>An app crashed somewhere because it loaded a garbage value from the stack. To show that problem the test is.a little longer than it would be if we only wanted to check " no tail + byval “.<br></blockquote><div><br></div><div>It is frowned upon to have tests which rely on LLVM's optimizers, it makes the resulting test susceptible to changes in code unrelated to clang.  I'd recommend we change the test to check for no tail + byval.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">
<div class="gmail-HOEnZb"><div class="gmail-h5">> I will let David decide on accepting that test or ask for improvement.<br>
><br>
> Regards,<br>
> Amjad<br>
><br>
><br>
> <a href="https://reviews.llvm.org/D22900" rel="noreferrer" target="_blank">https://reviews.llvm.org/D22900</a><br>
><br>
><br>
><br>
<br>
</div></div></blockquote></div><br></div></div>