[PATCH] D22900: Revert r244207 - Mark calls in thunk functions as tail-call optimization

Gerolf Hoflehner via cfe-commits cfe-commits at lists.llvm.org
Mon Aug 1 12:45:55 PDT 2016


I forwarded your the thread for the DSE path. Even my original intent for was a short-term bandage and by no means a long term fix. 

> On Aug 1, 2016, at 10:53 AM, Reid Kleckner <rnk at google.com> wrote:
> 
> rnk added a comment.
> 
> I think the correctness problem is in DSE. It should be safe for clang to put 'tail' here. The tail call does not capture the address of any local variables, but it does load from them via 'byval'.
> 
> 
> https://reviews.llvm.org/D22900
> 
> 
> 



More information about the cfe-commits mailing list