[PATCH] D146595: [clang] Add clang trampoline attribute

[David Blaikie via Phabricator via cfe-commits]

dblaikie added a comment.

Yeah, this seems a bit burdensome for both the frontend (encoding more information, the author has to duplicate the name of the implementation function so there's a risk they get out of sync?) and the debugger (is the debugger expected to jump directly and not call the implementation? Is it expected to continue into the implementation and break at the function? What if the function isn't called, or there's more than one call in the function, etc?)...

What about function overloading & namespaces? Is the debugger expected to figure out which `bar` function the DWARF/user is referring to?

I'd expect __attribute__((nodebug)) on the function, maybe, or some other single attribute that doesn't require retyping the implementation detail - __attribute__((debug_strep_transparent)) or the like?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146595