[PATCH] D58416: [llvm-cxxfilt] Split and demangle stdin input on certain non-alphanumerics.

Saleem Abdulrasool via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 19 16:34:58 PST 2019


compnerd added a comment.

Previously we would have emitted `Foo [[:space:]] .` right?



================
Comment at: llvm/tools/llvm-cxxfilt/llvm-cxxfilt.cpp:125
+    SplitStringDelims(Mangled, Words, IsLegalItaniumChar);
+    for (auto &Word : Words)
+      Result += demangle(OS, Word.first) + Word.second.str();
----------------
Shouldn't this be `const auto &`


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

https://reviews.llvm.org/D58416





More information about the llvm-commits mailing list