Revert "Swift: deduplicate accessors and params correctly"

This reverts commit bab4eee.

Author: redsun82

swift/extractor/mangler/SwiftMangler.cpp

@@ -65,17 +65,16 @@ SwiftMangledName SwiftMangler::visitValueDecl(const swift::ValueDecl* decl, bool
   auto ret = initMangled(decl);
   std::string name;
   llvm::raw_string_ostream oss{name};
-  oss << decl->getName();
+  decl->getName().print(oss);
   ret << name;
   if (decl->isStatic()) {
     ret << "|static";
   }
   return ret;
 }
 
-SwiftMangledName SwiftMangler::visitTypeDiscriminatedValueDecl(const swift::ValueDecl* decl,
-                                                               bool force) {
-  if (auto ret = visitValueDecl(decl, force)) {
+SwiftMangledName SwiftMangler::visitTypeDiscriminatedValueDecl(const swift::ValueDecl* decl) {
+  if (auto ret = visitValueDecl(decl)) {
     ret << fetch(decl->getInterfaceType()->getCanonicalType());
     return ret;
   }
@@ -86,13 +85,6 @@ SwiftMangledName SwiftMangler::visitAbstractFunctionDecl(const swift::AbstractFu
   return visitTypeDiscriminatedValueDecl(decl);
 }
 
-SwiftMangledName SwiftMangler::visitAccessorDecl(const swift::AccessorDecl* decl) {
-  std::string name;
-  llvm::raw_string_ostream oss{name};
-  decl->printUserFacingName(oss);
-  return visitTypeDiscriminatedValueDecl(decl, /*force=*/true) << '_' << name;
-}
-
 SwiftMangledName SwiftMangler::visitSubscriptDecl(const swift::SubscriptDecl* decl) {
   return visitTypeDiscriminatedValueDecl(decl);
 }
@@ -101,10 +93,6 @@ SwiftMangledName SwiftMangler::visitVarDecl(const swift::VarDecl* decl) {
   return visitTypeDiscriminatedValueDecl(decl);
 }
 
-SwiftMangledName SwiftMangler::visitParamDecl(const swift::ParamDecl* decl) {
-  return visitTypeDiscriminatedValueDecl(decl, /*force=*/true);
-}
-
 SwiftMangledName SwiftMangler::visitExtensionDecl(const swift::ExtensionDecl* decl) {
   if (decl->getDeclContext()->isLocalContext()) {
     return {};

swift/extractor/mangler/SwiftMangler.h

@@ -58,10 +58,8 @@ class SwiftMangler : private swift::TypeVisitor<SwiftMangler, SwiftMangledName>,
   SwiftMangledName visitModuleDecl(const swift::ModuleDecl* decl);
   SwiftMangledName visitExtensionDecl(const swift::ExtensionDecl* decl);
   SwiftMangledName visitAbstractFunctionDecl(const swift::AbstractFunctionDecl* decl);
-  SwiftMangledName visitAccessorDecl(const swift::AccessorDecl* decl);
   SwiftMangledName visitSubscriptDecl(const swift::SubscriptDecl* decl);
   SwiftMangledName visitVarDecl(const swift::VarDecl* decl);
-  SwiftMangledName visitParamDecl(const swift::ParamDecl* decl);
   SwiftMangledName visitAbstractTypeParamDecl(const swift::AbstractTypeParamDecl* decl);
   SwiftMangledName visitGenericTypeParamDecl(const swift::GenericTypeParamDecl* decl);
 
@@ -113,8 +111,7 @@ class SwiftMangler : private swift::TypeVisitor<SwiftMangler, SwiftMangledName>,
   unsigned int getExtensionIndex(const swift::ExtensionDecl* decl, const swift::Decl* parent);
   static SwiftMangledName initMangled(const swift::TypeBase* type);
   SwiftMangledName initMangled(const swift::Decl* decl);
-  SwiftMangledName visitTypeDiscriminatedValueDecl(const swift::ValueDecl* decl,
-                                                   bool force = false);
+  SwiftMangledName visitTypeDiscriminatedValueDecl(const swift::ValueDecl* decl);
 };
 
 // This implementation is indented for use in defining trap keys. In this case fetching gives