[PATCH] D65035: [DebugInfo] Don't emit incorrect descriptions of thunk params (PR42627)

[Vedant Kumar via Phabricator via llvm-commits]

vsk added a comment.

In D65035#1615585 <https://reviews.llvm.org/D65035#1615585>, @dblaikie wrote:

> Should we describe thunks at all? (gcc 8.1 doesn't)
>
> Interestingly gcc seems to include the thunk in the compile_unit address range, but doesn't have a subprogram for it...


That's a great question. That could save a little space, but it could also impair tools that treat dsyms as the source of truth for symbol information, like dtrace. I'd be open to it if the size savings were appreciable.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65035/new/

https://reviews.llvm.org/D65035