[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