[PATCH] D99981: [demangler] Support the new Rust mangling scheme (v0)

[James Henderson via Phabricator via llvm-commits]

jhenderson added a comment.

If you are looking for a full review of this, try adding specific people as reviewers who have worked on the demangling library. It would also make sense to split this change up into lots of small parts. For example, you could implement a small subset of the Rust demangling scheme, in a first patch, and expand on it in a series of subsequent patches. That'll make each patch easier to review. Connecting it up to llvm-cxxfilt could also be a separate patch from implementing the demangling code.



================
Comment at: llvm/test/tools/llvm-cxxfilt/rust.test:1
+RUN: llvm-cxxfilt -n  < %s | FileCheck --match-full-lines %s
+
----------------
llvm-cxxfilt tool tests are not the place to test the full library functionality.  - that should be tested in the unittests. This test should contain just enough to cover the behaviour implemented in llvm-cxxfilt itself.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99981