[PATCH] D111415: [Demangle] Add support for D simple qualified names
David Blaikie via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Nov 18 11:31:24 PST 2021
dblaikie added a comment.
Still seems a bit long in one patch/hard to verify that all the cases are covered by tests, etc. Could you break this up further - maybe implementing only the most basic, single 'parseIdentifier' case, then the iteration, then the anonymous symbols, then the lnames?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D111415/new/
https://reviews.llvm.org/D111415
More information about the llvm-commits
mailing list