[PATCH] D50828: Add profiling and canonicalization support to demangler nodes. No functionality change intended.

[Richard Smith - zygoloid via Phabricator via llvm-commits]

rsmith added a comment.

I'm going to try to split this patch so it adds a generic visitation mechanism to the demangler AST, and introduce a separate layer that includes Support and builds profiling off the visitation mechanism. That seems like the only way out that meets everyone's acceptance criteria. However, I don't see any way to do that without moving all of the demangler to a public header in include/llvm (since the whole thing will need to be included from both the Demangle/ demangler and the profiling demangler). That header would only be included (and the templates in it instantiated) in two places in LLVM, but still, it'd be a 5000-line header file. If people are strongly opposed to that approach, please shout now!


Repository:
  rL LLVM

https://reviews.llvm.org/D50828