[PATCH] D114892: [llvm-dwarfdump] Do not print preceding :: for local types

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 3 00:37:28 PST 2021


jhenderson accepted this revision.
jhenderson added a comment.

In D114892#3167540 <https://reviews.llvm.org/D114892#3167540>, @dblaikie wrote:

> In D114892#3166220 <https://reviews.llvm.org/D114892#3166220>, @jhenderson wrote:
>
>> The patch is more about testing generic DebugInfo behaviour, rather than anything specific to llvm-dwarfdump. I wonder if the test would make more sense there? No strong feelings on that though.
>
> This comes back to the complication of debug info testing - llvm/test/DebugInfo has ended up as more a place for testing CodeGen/AsmPrinter - debug info generation, though, yeah, that's non-obvious. @probinson did point this out a while back that maybe the LLVM codegen tests should be moved into a different directory so llvm/test/DebugInfo could be used for testing libDebugInfo - and generally it was decided not to do that. I may have second thoughts on that, but generally I'd prefer pure libDebugInfoDWARF testing happen in llvm/test/tools/llvm-dwarfdump (& llvm/test/tools/llvm-symbolizer, etc) for now/these days unless we're going to move the codegen testing out of llvm/test/DebugInfo.

Okay, fair enough. Personal opinion is that there should be a one-to-one mapping between libraries/tools and test directories, which matches @probinson's points, but llvm-dwarfdump is as good as anywhere for this now, I guess (even ignoring the pre-existing test already mentioned).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114892



More information about the llvm-commits mailing list