[PATCH] D153869: [DWARFv5][DWARFLinker] avoid stripping template names for .debug_names.

Felipe de Azevedo Piovezan via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 12 03:01:28 PDT 2023


fdeazeve added a comment.

> Probably we also need to change clang to use the same behavior. i.e. add record with stripped template names into .debug_names. So that lldb can rely on this on non-darwin platform.

Yup, I would like to make this a "non-dsym" feature as well. As I mentioned above, it doesn't seem right that the linker is trying to improve the quality of debug info, instead of only linking/optimizing it; the compiler should be the one generating all the required information.

... I've just realized that my links weren't pasted properly in the other message, so here they are for completeness. I think the relevant discussion for us is [1]

  [1]: https://github.com/llvm/llvm-project/issues/58362
  [2]: https://discourse.llvm.org/t/dwarf-using-simplified-template-names/58417/11
  [3]: https://reviews.llvm.org/rG38c09ea2d279eddddabe3602e2002f8cdfcc5380
  [4]: https://reviews.llvm.org/D134378

> I will create a patch restoring old dsymutil behavior tomorrow.

Thank you!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153869



More information about the llvm-commits mailing list