[PATCH] D104358: [Demangle][Rust] Parse dot suffix

[David Blaikie via Phabricator via llvm-commits]

dblaikie added inline comments.


================
Comment at: llvm/lib/Demangle/RustDemangle.cpp:121-128
+  // An extension allowing an arbitrary dot suffix.
+  if (consumeIf('.')) {
+    auto Suffix = Input.substr(Position);
+    Position += Suffix.size();
+    print(" (");
+    print(Suffix);
+    print(")");
----------------
Is this similar to the Itanium implementation? Or would it be possible/reasonable/simpler (maybe not, I don't know) to strip the '.' before doing any parsing at all? (or are there cases where the rust mangling can include '.' and so finding the trailing '.' must be a context-sensitive operation)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104358