[PATCH] D47248: [demangler] Add ItaniumPartialDemangler::isCtorOrDtor

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 23 16:35:19 PDT 2018


MaskRay marked 2 inline comments as done.
MaskRay added inline comments.


================
Comment at: lib/Demangle/ItaniumDemangle.cpp:5271
+  while (N) {
+    switch (N->getKind()) {
+    default:
----------------
erik.pilkington wrote:
> No reason to include DtorName or QualifiedName, those can only appear in expressions (via an <unresolved-name>) so they should be unreachable. 
Done.

Thanks. I see, these cases are covered by `getFunctionBaseName` above.


Repository:
  rL LLVM

https://reviews.llvm.org/D47248





More information about the llvm-commits mailing list