Skip to content

add range tracking when demangling a name #82298

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
232 changes: 225 additions & 7 deletions include/swift/Demangling/Demangle.h
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,11 @@
#ifndef SWIFT_DEMANGLING_DEMANGLE_H
#define SWIFT_DEMANGLING_DEMANGLE_H

#include "swift/Demangling/Demangle.h"
#include "swift/Demangling/Errors.h"
#include "swift/Demangling/ManglingFlavor.h"
#include "swift/Demangling/NamespaceMacros.h"
#include "swift/Strings.h"
#include "llvm/ADT/StringRef.h"
#include "llvm/Support/Compiler.h"

Expand Down Expand Up @@ -99,6 +101,7 @@ struct DemangleOptions {

class Node;
using NodePointer = Node *;
class NodePrinter;

enum class FunctionSigSpecializationParamKind : unsigned {
// Option Flags use bits 0-5. This give us 6 bits implying 64 entries to
Expand Down Expand Up @@ -234,6 +237,18 @@ class Node {
public:
Kind getKind() const { return NodeKind; }

bool shouldTrackNameRange() const {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where is this used?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

switch (getKind()) {
case Kind::Function:
case Kind::Constructor:
case Kind::Allocator:
case Kind::ExplicitClosure:
return true;
default:
return false;
}
}

bool isSimilarTo(const Node *other) const {
if (NodeKind != other->NodeKind
|| NodePayloadKind != other->NodePayloadKind)
Expand Down Expand Up @@ -417,6 +432,10 @@ bool isOldFunctionTypeMangling(llvm::StringRef mangledName);

class Demangler;

class DemanglerPrinter;

class TrackingDemanglerPrinter;
Comment on lines +435 to +437
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
class DemanglerPrinter;
class TrackingDemanglerPrinter;

Don't think these are needed

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.


/// The demangler context.
///
/// It owns the allocated nodes which are created during demangling.
Expand Down Expand Up @@ -465,16 +484,26 @@ class Context {
/// The lifetime of the returned node tree ends with the lifetime of the
/// context or with a call of clear().
NodePointer demangleTypeAsNode(llvm::StringRef MangledName);

/// Demangle the given symbol and return the readable name.
///
/// \param MangledName The mangled symbol string, which start a mangling
/// prefix: _T, _T0, $S, _$S.
///
/// \returns The demangled string.
std::string demangleSymbolAsString(
llvm::StringRef MangledName,
const DemangleOptions &Options = DemangleOptions());
std::string
demangleSymbolAsString(llvm::StringRef MangledName,
const DemangleOptions &Options = DemangleOptions());

/// Demangle the given symbol.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lets briefly describe what the intended usage of this overload is

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

///
/// \param MangledName The mangled symbol string, which start a mangling
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
/// \param MangledName The mangled symbol string, which start a mangling
/// \param MangledName The mangled symbol string, which start with a mangling

?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not sure where this came from, probably a bad copy paste. Removed in:

/// prefix: _T, _T0, $S, _$S.
/// \param printer The NodePrinter that will be used to demangle the symbol.
///
/// \returns The demangled string.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This function doesn't return anything though.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

void demangleSymbolAsString(llvm::StringRef MangledName,
NodePrinter *printer);

/// Demangle the given type and return the readable name.
///
Expand Down Expand Up @@ -533,6 +562,18 @@ std::string
demangleSymbolAsString(const char *mangledName, size_t mangledNameLength,
const DemangleOptions &options = DemangleOptions());

/// Standalone utility function to demangle the given symbol as string.
///
/// If performance is an issue when demangling multiple symbols,
/// Context::demangleSymbolAsString should be used instead.
/// \param mangledName The mangled name string pointer.
/// \param mangledNameLength The length of the mangledName string.
/// \param printer The NodePrinter that will be used to demangle the symbol.
///
/// \returns The demangled string.
void demangleSymbolAsString(const char *mangledName, size_t mangledNameLength,
NodePrinter *printer);

/// Standalone utility function to demangle the given symbol as string.
///
/// If performance is an issue when demangling multiple symbols,
Expand All @@ -545,7 +586,7 @@ demangleSymbolAsString(const std::string &mangledName,
return demangleSymbolAsString(mangledName.data(), mangledName.size(),
options);
}

/// Standalone utility function to demangle the given symbol as string.
///
/// If performance is an issue when demangling multiple symbols,
Expand All @@ -555,8 +596,20 @@ demangleSymbolAsString(const std::string &mangledName,
inline std::string
demangleSymbolAsString(llvm::StringRef MangledName,
const DemangleOptions &Options = DemangleOptions()) {
return demangleSymbolAsString(MangledName.data(),
MangledName.size(), Options);
return demangleSymbolAsString(MangledName.data(), MangledName.size(),
Options);
}

/// Standalone utility function to demangle the given symbol as string.
///
/// If performance is an issue when demangling multiple symbols,
/// Context::demangleSymbolAsString should be used instead.
/// \param MangledName The mangled name string.
///
/// \returns The demangled string.
inline void demangleSymbolAsString(llvm::StringRef MangledName,
NodePrinter *printer) {
demangleSymbolAsString(MangledName.data(), MangledName.size(), printer);
}

/// Standalone utility function to demangle the given type as string.
Expand Down Expand Up @@ -732,6 +785,15 @@ ManglingErrorOr<const char *> mangleNodeAsObjcCString(NodePointer node,
std::string nodeToString(NodePointer Root,
const DemangleOptions &Options = DemangleOptions());

/// Transform the node structure to a string.
///
/// \endcode
///
/// \param Root A pointer to a parse tree generated by the demangler.
/// \param Printer A NodePrinter used to pretty print the demangled Node.
///
void nodeToString(NodePointer Root, NodePrinter *Printer);

/// Transforms a mangled key path accessor thunk helper
/// into the identfier/subscript that would be used to invoke it in swift code.
std::string keyPathSourceString(const char *MangledName,
Expand Down Expand Up @@ -777,11 +839,14 @@ class DemanglerPrinter {

llvm::StringRef getStringRef() const { return Stream; }

size_t getStreamLength() { return Stream.length(); }

/// Shrinks the buffer.
void resetSize(size_t toPos) {
assert(toPos <= Stream.size());
Stream.resize(toPos);
}

private:
std::string Stream;
};
Expand Down Expand Up @@ -818,6 +883,159 @@ std::string mangledNameForTypeMetadataAccessor(
llvm::StringRef moduleName, llvm::StringRef typeName, Node::Kind typeKind,
Mangle::ManglingFlavor Flavor = Mangle::ManglingFlavor::Default);

/// Base class for printing a Swift demangled node tree.
///
/// NodePrinter is used to convert demangled Swift symbol nodes into
/// human-readable string representations. It handles formatting, indentation,
/// and Swift-specific syntax.
///
/// The virtual methods in this class are meant to be overriden to allow
/// external consumers (e.g lldb) to track the ranges of components of the
/// demangled name.
class NodePrinter {
protected:
DemanglerPrinter Printer;
DemangleOptions Options;
bool SpecializationPrefixPrinted = false;
bool isValid = true;

public:
NodePrinter(DemangleOptions options) : Options(options) {}

virtual ~NodePrinter() {}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
virtual ~NodePrinter() {}
virtual ~NodePrinter() = default

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.


void printRoot(NodePointer root) {
isValid = true;
print(root, 0);
}

std::string takeString() {
if (isValid)
return std::move(Printer).str();
return "";
}

protected:
static const unsigned MaxDepth = 768;

size_t getStreamLength() { return Printer.getStreamLength(); }

/// Called when the node tree in valid.
///
/// The demangler already catches most error cases and mostly produces valid
/// node trees. But some cases are difficult to catch in the demangler and
/// instead the NodePrinter bails.
void setInvalid() { isValid = false; }

void printChildren(Node::iterator begin, Node::iterator end, unsigned depth,
const char *sep = nullptr);

void printChildren(NodePointer Node, unsigned depth,
const char *sep = nullptr);

NodePointer getFirstChildOfKind(NodePointer Node, Node::Kind kind);

void printBoundGenericNoSugar(NodePointer Node, unsigned depth);

void printOptionalIndex(NodePointer node);

static bool isSwiftModule(NodePointer node) {
return (node->getKind() == Node::Kind::Module &&
node->getText() == STDLIB_NAME);
}

static bool isIdentifier(NodePointer node, StringRef desired) {
return (node->getKind() == Node::Kind::Identifier &&
node->getText() == desired);
}

bool printContext(NodePointer Context);

enum class SugarType {
None,
Optional,
ImplicitlyUnwrappedOptional,
Array,
Dictionary
};

enum class TypePrinting { NoType, WithColon, FunctionStyle };

/// Determine whether this is a "simple" type, from the type-simple
/// production.
bool isSimpleType(NodePointer Node);

void printWithParens(NodePointer type, unsigned depth);

SugarType findSugar(NodePointer Node);

void printBoundGeneric(NodePointer Node, unsigned depth);

NodePointer getChildIf(NodePointer Node, Node::Kind Kind);

virtual void printFunctionParameters(NodePointer LabelList,
NodePointer ParameterType,
unsigned depth, bool showTypes);

void printFunctionType(NodePointer LabelList, NodePointer node,
unsigned depth);

void printImplFunctionType(NodePointer fn, unsigned depth);

void printGenericSignature(NodePointer Node, unsigned depth);

void printFunctionSigSpecializationParams(NodePointer Node, unsigned depth);

void printSpecializationPrefix(NodePointer node, StringRef Description,
unsigned depth,
StringRef ParamPrefix = StringRef());

/// The main big print function.
NodePointer print(NodePointer Node, unsigned depth,
bool asPrefixContext = false);

NodePointer printAbstractStorage(NodePointer Node, unsigned depth,
bool asPrefixContent, StringRef ExtraName);

/// Utility function to print entities.
///
/// \param Entity The entity node to print
/// \param depth The depth in the print() call tree.
/// \param asPrefixContext Should the entity printed as a context which as a
/// prefix to another entity, e.g. the Abc in Abc.def()
/// \param TypePr How should the type of the entity be printed, if at all.
/// E.g. with a colon for properties or as a function type.
/// \param hasName Does the entity has a name, e.g. a function in contrast to
/// an initializer.
/// \param ExtraName An extra name added to the entity name (if any).
/// \param ExtraIndex An extra index added to the entity name (if any),
/// e.g. closure #1
/// \param OverwriteName If non-empty, print this name instead of the one
/// provided by the node. Gets printed even if hasName is false.
/// \return If a non-null node is returned it's a context which must be
/// printed in postfix-form after the entity: "<entity> in <context>".
NodePointer printEntity(NodePointer Entity, unsigned depth,
bool asPrefixContext, TypePrinting TypePr,
bool hasName, StringRef ExtraName = "",
int ExtraIndex = -1, StringRef OverwriteName = "");

virtual void printFunctionName(bool hasName, llvm::StringRef &OverwriteName,
llvm::StringRef &ExtraName, bool MultiWordName,
int &ExtraIndex,
swift::Demangle::NodePointer Entity,
unsigned int depth);

/// Print the type of an entity.
///
/// \param Entity The entity.
/// \param type The type of the entity.
/// \param genericFunctionTypeList If not null, the generic argument types
/// which is printed in the generic signature.
/// \param depth The depth in the print() call tree.
void printEntityType(NodePointer Entity, NodePointer type,
NodePointer genericFunctionTypeList, unsigned depth);
};

SWIFT_END_INLINE_NAMESPACE
} // end namespace Demangle
} // end namespace swift
Expand Down
13 changes: 13 additions & 0 deletions lib/Demangling/Context.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -66,6 +66,12 @@ std::string Context::demangleSymbolAsString(llvm::StringRef MangledName,
return demangling;
}

void Context::demangleSymbolAsString(llvm::StringRef MangledName,
NodePrinter *printer) {
NodePointer root = demangleSymbolAsNode(MangledName);
nodeToString(root, printer);
}

std::string Context::demangleTypeAsString(llvm::StringRef MangledName,
const DemangleOptions &Options) {
NodePointer root = demangleTypeAsNode(MangledName);
Expand Down Expand Up @@ -275,6 +281,13 @@ std::string demangleSymbolAsString(const char *MangledName,
Options);
}

void demangleSymbolAsString(const char *MangledName, size_t MangledNameLength,
NodePrinter *printer) {
Context Ctx;
return Ctx.demangleSymbolAsString(StringRef(MangledName, MangledNameLength),
printer);
}

std::string demangleTypeAsString(const char *MangledName,
size_t MangledNameLength,
const DemangleOptions &Options) {
Expand Down
Loading