<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On Jul 1, 2013, at 10:37 AM, Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">On Mon, Jul 1, 2013 at 10:28 AM, Quentin Colombet <<a href="mailto:qcolombet@apple.com">qcolombet@apple.com</a>> wrote:<br><blockquote type="cite">Hi Tim,<br><br>The patch LGTM with one question:<br>Do you have any performance number to see the impact of this patch?<br><br>Indeed, tail calls are generally faster, but I am wondering why it was not<br>enabled for non-iOS targets before your patch. Hence, having some evidences<br>that it does not have weird side effects would be nice.<br><br></blockquote><br>My guess here is likely testing to be honest, i.e. not wanting to turn<br>on an untested code path by default.<br><br></div></blockquote><div dir="auto"><br></div><div dir="auto">Yes.</div><div dir="auto"><br></div><div dir="auto">-Jim</div><br><blockquote type="cite"><div style="font-size: 12px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px;">I would like to see any performance data though. :)<br><br>-eric<br><br><blockquote type="cite">Thanks,<br><br>-Quentin<br><br>On Jun 30, 2013, at 3:12 AM, Tim Northover <<a href="mailto:t.p.northover@gmail.com">t.p.northover@gmail.com</a>> wrote:<br><br>Hi,<br><br>The attached patch enables tail calls for every target that hasn't<br>explicitly been marked as bad (iOS < 5.0). This amounts to any ELF<br>target and being more liberal on some ghost MachO targets (e.g.<br>arm-apple-darwin9) that no-one probably cares about.<br><br>I think the ELF side is justified because the required linker support<br>is there in the ARM ELF specification so anywhere it doesn't work has<br>a broken linker. I've also tested it on Linux and our MCJIT dynamic<br>loader with no issues.<br><br>Does anyone object to the change?<br><br>Cheers.<br><br>Tim.<br><enable-tail.diff>_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits<br><br><br><br>_______________________________________________<br>llvm-commits mailing list<br>llvm-commits@cs.uiuc.edu<br>http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits<br><br></blockquote>_______________________________________________<br>llvm-commits mailing list<br><a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br><a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a></div></blockquote></div><br></body></html>