[PATCH] D22900: Revert r244207 - Mark calls in thunk functions as tail-call optimization
Amjad Aboud via cfe-commits
cfe-commits at lists.llvm.org
Sun Jul 31 01:46:34 PDT 2016
aaboud added a comment.
> 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.
>
> In any case, correctness must always trump all else. Reverting to green should take precedence over a QoI bug like PR24235.
I agree to the revert, though I am not sure about the new test, it looks too complected, especially the command line.
I will let David decide on accepting that test or ask for improvement.
Regards,
Amjad
https://reviews.llvm.org/D22900
More information about the cfe-commits
mailing list