[PATCH] D118486: [demangler] Improve ->* & .* demangling

Bruno Cardoso Lopes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 7 14:42:34 PST 2022


bruno accepted this revision.
bruno added a comment.
This revision is now accepted and ready to land.

> The right solution to the paren problem is to add some notion of precedence (and associativity) to Nodes, but that's a larger change that would become simpler once the refactoring I'm doing is completed.

Is this worth putting in a FIXME somewhere as part of this patch? Or is this being tracked elsewhere?

Other than that, LGTM.


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

https://reviews.llvm.org/D118486



More information about the llvm-commits mailing list