[Lldb-commits] [PATCH] D115308: [LLDB][DWARF] Fix duplicate TypeSP in type list
Pavel Labath via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Wed Dec 22 04:13:30 PST 2021
labath accepted this revision.
labath added a comment.
This revision is now accepted and ready to land.
Since you're simply removing code, I don't think that we need to insist on new test cases -- not breaking existing ones should suffice. That said, if you know of a way to test this via the `image dump symfile` command, I would definitely encourage you to do so.
And yes, we need to figure out what to do with lldb-test. Maybe we should just rewrite `ParseTypes` (used only in tests) to do the parsing in a different way...
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D115308/new/
https://reviews.llvm.org/D115308
More information about the lldb-commits
mailing list