[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