[PATCH] D28075: MergeFunctions: Preserve debug info in thunks, under option -mergefunc-preserve-debug-info

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 6 13:57:25 PST 2017


aprantl added a comment.

In https://reviews.llvm.org/D28075#638014, @appcs wrote:

>




>   What should -mergefunc-preserve-debug-info do about the thunk's call to the shared implementation?
>   Mark as tail call or not? The existing -mergefunc behaviour is to mark it as a tail call. We could leave it as is,
>   unless someone specifically asks for a change under -mergefunc-preserve-debug-info; would that be OK?

I think marking it as a tail call makes most sense.


https://reviews.llvm.org/D28075





More information about the llvm-commits mailing list