Skip to content
Merged
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
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ class LLVM_ABI AppendingTypeTableBuilder : public TypeCollection {

public:
explicit AppendingTypeTableBuilder(BumpPtrAllocator &Storage);
~AppendingTypeTableBuilder();
~AppendingTypeTableBuilder() override;

// TypeCollection overrides
std::optional<TypeIndex> getFirst() override;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -47,7 +47,7 @@ class LLVM_ABI GlobalTypeTableBuilder : public TypeCollection {

public:
explicit GlobalTypeTableBuilder(BumpPtrAllocator &Storage);
~GlobalTypeTableBuilder();
~GlobalTypeTableBuilder() override;

// TypeCollection overrides
std::optional<TypeIndex> getFirst() override;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ class LLVM_ABI MergingTypeTableBuilder : public TypeCollection {

public:
explicit MergingTypeTableBuilder(BumpPtrAllocator &Storage);
~MergingTypeTableBuilder();
~MergingTypeTableBuilder() override;

// TypeCollection overrides
std::optional<TypeIndex> getFirst() override;
Expand Down
2 changes: 1 addition & 1 deletion llvm/include/llvm/DebugInfo/GSYM/GsymContext.h
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ class GsymReader;
class GsymContext : public DIContext {
public:
GsymContext(std::unique_ptr<GsymReader> Reader);
~GsymContext();
~GsymContext() override;

GsymContext(GsymContext &) = delete;
GsymContext &operator=(GsymContext &) = delete;
Expand Down
2 changes: 1 addition & 1 deletion llvm/include/llvm/DebugInfo/LogicalView/Core/LVElement.h
Original file line number Diff line number Diff line change
Expand Up @@ -143,7 +143,7 @@ class LLVM_ABI LVElement : public LVObject {
VirtualityCode(0) {}
LVElement(const LVElement &) = delete;
LVElement &operator=(const LVElement &) = delete;
virtual ~LVElement() = default;
~LVElement() override = default;

LVSubclassID getSubclassID() const { return SubclassID; }

Expand Down
6 changes: 3 additions & 3 deletions llvm/include/llvm/DebugInfo/LogicalView/Core/LVLine.h
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,7 @@ class LLVM_ABI LVLine : public LVElement {
}
LVLine(const LVLine &) = delete;
LVLine &operator=(const LVLine &) = delete;
virtual ~LVLine() = default;
~LVLine() override = default;

static bool classof(const LVElement *Element) {
return Element->getSubclassID() == LVSubclassID::LV_LINE;
Expand Down Expand Up @@ -117,7 +117,7 @@ class LLVM_ABI LVLineDebug final : public LVLine {
LVLineDebug() : LVLine() { setIsLineDebug(); }
LVLineDebug(const LVLineDebug &) = delete;
LVLineDebug &operator=(const LVLineDebug &) = delete;
~LVLineDebug() = default;
~LVLineDebug() override = default;

// Additional line information. It includes attributes that describes
// states in the machine instructions (basic block, end prologue, etc).
Expand All @@ -142,7 +142,7 @@ class LLVM_ABI LVLineAssembler final : public LVLine {
LVLineAssembler() : LVLine() { setIsLineAssembler(); }
LVLineAssembler(const LVLineAssembler &) = delete;
LVLineAssembler &operator=(const LVLineAssembler &) = delete;
~LVLineAssembler() = default;
~LVLineAssembler() override = default;

// Print blanks as the line number.
std::string noLineAsString(bool ShowZero) const override {
Expand Down
4 changes: 2 additions & 2 deletions llvm/include/llvm/DebugInfo/LogicalView/Core/LVLocation.h
Original file line number Diff line number Diff line change
Expand Up @@ -100,7 +100,7 @@ class LLVM_ABI LVLocation : public LVObject {
LVLocation() : LVObject() { setIsLocation(); }
LVLocation(const LVLocation &) = delete;
LVLocation &operator=(const LVLocation &) = delete;
virtual ~LVLocation() = default;
~LVLocation() override = default;

PROPERTY(Property, IsAddressRange);
PROPERTY(Property, IsBaseClassOffset);
Expand Down Expand Up @@ -171,7 +171,7 @@ class LLVM_ABI LVLocationSymbol final : public LVLocation {
LVLocationSymbol() : LVLocation() {}
LVLocationSymbol(const LVLocationSymbol &) = delete;
LVLocationSymbol &operator=(const LVLocationSymbol &) = delete;
~LVLocationSymbol() = default;
~LVLocationSymbol() override = default;

void addObject(LVAddress LowPC, LVAddress HighPC, LVUnsigned SectionOffset,
uint64_t LocDescOffset) override;
Expand Down
2 changes: 1 addition & 1 deletion llvm/include/llvm/DebugInfo/LogicalView/Core/LVRange.h
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ class LLVM_ABI LVRange final : public LVObject {
LVRange() : LVObject(), RangesTree(Allocator) {}
LVRange(const LVRange &) = delete;
LVRange &operator=(const LVRange &) = delete;
~LVRange() = default;
~LVRange() override = default;

void addEntry(LVScope *Scope, LVAddress LowerAddress, LVAddress UpperAddress);
void addEntry(LVScope *Scope);
Expand Down
28 changes: 14 additions & 14 deletions llvm/include/llvm/DebugInfo/LogicalView/Core/LVScope.h
Original file line number Diff line number Diff line change
Expand Up @@ -153,7 +153,7 @@ class LLVM_ABI LVScope : public LVElement {
}
LVScope(const LVScope &) = delete;
LVScope &operator=(const LVScope &) = delete;
virtual ~LVScope() = default;
~LVScope() override = default;

static bool classof(const LVElement *Element) {
return Element->getSubclassID() == LVSubclassID::LV_SCOPE;
Expand Down Expand Up @@ -349,7 +349,7 @@ class LLVM_ABI LVScopeAggregate final : public LVScope {
LVScopeAggregate() : LVScope() {}
LVScopeAggregate(const LVScopeAggregate &) = delete;
LVScopeAggregate &operator=(const LVScopeAggregate &) = delete;
~LVScopeAggregate() = default;
~LVScopeAggregate() override = default;

// DW_AT_specification, DW_AT_abstract_origin.
LVScope *getReference() const override { return Reference; }
Expand Down Expand Up @@ -387,7 +387,7 @@ class LLVM_ABI LVScopeAlias final : public LVScope {
}
LVScopeAlias(const LVScopeAlias &) = delete;
LVScopeAlias &operator=(const LVScopeAlias &) = delete;
~LVScopeAlias() = default;
~LVScopeAlias() override = default;

// Returns true if current scope is logically equal to the given 'Scope'.
bool equals(const LVScope *Scope) const override;
Expand All @@ -401,7 +401,7 @@ class LLVM_ABI LVScopeArray final : public LVScope {
LVScopeArray() : LVScope() { setIsArray(); }
LVScopeArray(const LVScopeArray &) = delete;
LVScopeArray &operator=(const LVScopeArray &) = delete;
~LVScopeArray() = default;
~LVScopeArray() override = default;

void resolveExtra() override;

Expand Down Expand Up @@ -513,7 +513,7 @@ class LLVM_ABI LVScopeCompileUnit final : public LVScope {
}
LVScopeCompileUnit(const LVScopeCompileUnit &) = delete;
LVScopeCompileUnit &operator=(const LVScopeCompileUnit &) = delete;
~LVScopeCompileUnit() = default;
~LVScopeCompileUnit() override = default;

LVScope *getCompileUnitParent() const override {
return static_cast<LVScope *>(const_cast<LVScopeCompileUnit *>(this));
Expand Down Expand Up @@ -643,7 +643,7 @@ class LLVM_ABI LVScopeEnumeration final : public LVScope {
LVScopeEnumeration() : LVScope() { setIsEnumeration(); }
LVScopeEnumeration(const LVScopeEnumeration &) = delete;
LVScopeEnumeration &operator=(const LVScopeEnumeration &) = delete;
~LVScopeEnumeration() = default;
~LVScopeEnumeration() override = default;

// Returns true if current scope is logically equal to the given 'Scope'.
bool equals(const LVScope *Scope) const override;
Expand All @@ -658,7 +658,7 @@ class LLVM_ABI LVScopeFormalPack final : public LVScope {
LVScopeFormalPack() : LVScope() { setIsTemplatePack(); }
LVScopeFormalPack(const LVScopeFormalPack &) = delete;
LVScopeFormalPack &operator=(const LVScopeFormalPack &) = delete;
~LVScopeFormalPack() = default;
~LVScopeFormalPack() override = default;

// Returns true if current scope is logically equal to the given 'Scope'.
bool equals(const LVScope *Scope) const override;
Expand All @@ -676,7 +676,7 @@ class LLVM_ABI LVScopeFunction : public LVScope {
LVScopeFunction() : LVScope() {}
LVScopeFunction(const LVScopeFunction &) = delete;
LVScopeFunction &operator=(const LVScopeFunction &) = delete;
virtual ~LVScopeFunction() = default;
~LVScopeFunction() override = default;

// DW_AT_specification, DW_AT_abstract_origin.
LVScope *getReference() const override { return Reference; }
Expand Down Expand Up @@ -728,7 +728,7 @@ class LLVM_ABI LVScopeFunctionInlined final : public LVScopeFunction {
LVScopeFunctionInlined() : LVScopeFunction() { setIsInlinedFunction(); }
LVScopeFunctionInlined(const LVScopeFunctionInlined &) = delete;
LVScopeFunctionInlined &operator=(const LVScopeFunctionInlined &) = delete;
~LVScopeFunctionInlined() = default;
~LVScopeFunctionInlined() override = default;

uint32_t getDiscriminator() const override { return Discriminator; }
void setDiscriminator(uint32_t Value) override {
Expand Down Expand Up @@ -767,7 +767,7 @@ class LLVM_ABI LVScopeFunctionType final : public LVScopeFunction {
LVScopeFunctionType() : LVScopeFunction() { setIsFunctionType(); }
LVScopeFunctionType(const LVScopeFunctionType &) = delete;
LVScopeFunctionType &operator=(const LVScopeFunctionType &) = delete;
~LVScopeFunctionType() = default;
~LVScopeFunctionType() override = default;

void resolveExtra() override;
};
Expand All @@ -781,7 +781,7 @@ class LLVM_ABI LVScopeModule final : public LVScope {
}
LVScopeModule(const LVScopeModule &) = delete;
LVScopeModule &operator=(const LVScopeModule &) = delete;
~LVScopeModule() = default;
~LVScopeModule() override = default;

// Returns true if current scope is logically equal to the given 'Scope'.
bool equals(const LVScope *Scope) const override;
Expand All @@ -797,7 +797,7 @@ class LLVM_ABI LVScopeNamespace final : public LVScope {
LVScopeNamespace() : LVScope() { setIsNamespace(); }
LVScopeNamespace(const LVScopeNamespace &) = delete;
LVScopeNamespace &operator=(const LVScopeNamespace &) = delete;
~LVScopeNamespace() = default;
~LVScopeNamespace() override = default;

// Access DW_AT_extension reference.
LVScope *getReference() const override { return Reference; }
Expand Down Expand Up @@ -827,7 +827,7 @@ class LLVM_ABI LVScopeRoot final : public LVScope {
LVScopeRoot() : LVScope() { setIsRoot(); }
LVScopeRoot(const LVScopeRoot &) = delete;
LVScopeRoot &operator=(const LVScopeRoot &) = delete;
~LVScopeRoot() = default;
~LVScopeRoot() override = default;

StringRef getFileFormatName() const {
return getStringPool().getString(FileFormatNameIndex);
Expand Down Expand Up @@ -859,7 +859,7 @@ class LLVM_ABI LVScopeTemplatePack final : public LVScope {
LVScopeTemplatePack() : LVScope() { setIsTemplatePack(); }
LVScopeTemplatePack(const LVScopeTemplatePack &) = delete;
LVScopeTemplatePack &operator=(const LVScopeTemplatePack &) = delete;
~LVScopeTemplatePack() = default;
~LVScopeTemplatePack() override = default;

// Returns true if current scope is logically equal to the given 'Scope'.
bool equals(const LVScope *Scope) const override;
Expand Down
2 changes: 1 addition & 1 deletion llvm/include/llvm/DebugInfo/LogicalView/Core/LVSymbol.h
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,7 @@ class LLVM_ABI LVSymbol final : public LVElement {
}
LVSymbol(const LVSymbol &) = delete;
LVSymbol &operator=(const LVSymbol &) = delete;
~LVSymbol() = default;
~LVSymbol() override = default;

static bool classof(const LVElement *Element) {
return Element->getSubclassID() == LVSubclassID::LV_SYMBOL;
Expand Down
12 changes: 6 additions & 6 deletions llvm/include/llvm/DebugInfo/LogicalView/Core/LVType.h
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ class LLVM_ABI LVType : public LVElement {
LVType() : LVElement(LVSubclassID::LV_TYPE) { setIsType(); }
LVType(const LVType &) = delete;
LVType &operator=(const LVType &) = delete;
virtual ~LVType() = default;
~LVType() override = default;

static bool classof(const LVElement *Element) {
return Element->getSubclassID() == LVSubclassID::LV_TYPE;
Expand Down Expand Up @@ -157,7 +157,7 @@ class LLVM_ABI LVTypeDefinition final : public LVType {
}
LVTypeDefinition(const LVTypeDefinition &) = delete;
LVTypeDefinition &operator=(const LVTypeDefinition &) = delete;
~LVTypeDefinition() = default;
~LVTypeDefinition() override = default;

// Return the underlying type for a type definition.
LVElement *getUnderlyingType() override;
Expand All @@ -183,7 +183,7 @@ class LLVM_ABI LVTypeEnumerator final : public LVType {
}
LVTypeEnumerator(const LVTypeEnumerator &) = delete;
LVTypeEnumerator &operator=(const LVTypeEnumerator &) = delete;
~LVTypeEnumerator() = default;
~LVTypeEnumerator() override = default;

// Process the values for a DW_TAG_enumerator.
StringRef getValue() const override {
Expand All @@ -206,7 +206,7 @@ class LLVM_ABI LVTypeImport final : public LVType {
LVTypeImport() : LVType() { setIncludeInPrint(); }
LVTypeImport(const LVTypeImport &) = delete;
LVTypeImport &operator=(const LVTypeImport &) = delete;
~LVTypeImport() = default;
~LVTypeImport() override = default;

// Returns true if current type is logically equal to the given 'Type'.
bool equals(const LVType *Type) const override;
Expand All @@ -223,7 +223,7 @@ class LLVM_ABI LVTypeParam final : public LVType {
LVTypeParam();
LVTypeParam(const LVTypeParam &) = delete;
LVTypeParam &operator=(const LVTypeParam &) = delete;
~LVTypeParam() = default;
~LVTypeParam() override = default;

// Template parameter value.
StringRef getValue() const override {
Expand Down Expand Up @@ -256,7 +256,7 @@ class LLVM_ABI LVTypeSubrange final : public LVType {
}
LVTypeSubrange(const LVTypeSubrange &) = delete;
LVTypeSubrange &operator=(const LVTypeSubrange &) = delete;
~LVTypeSubrange() = default;
~LVTypeSubrange() override = default;

int64_t getCount() const override {
return getIsSubrangeCount() ? LowerBound : 0;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -192,7 +192,7 @@ class LVBinaryReader : public LVReader {
: LVReader(Filename, FileFormatName, W, BinaryType) {}
LVBinaryReader(const LVBinaryReader &) = delete;
LVBinaryReader &operator=(const LVBinaryReader &) = delete;
virtual ~LVBinaryReader() = default;
~LVBinaryReader() override = default;

void addInlineeLines(LVScope *Scope, LVLines &Lines) {
CUInlineeLines.emplace(Scope, std::make_unique<LVLines>(std::move(Lines)));
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -200,7 +200,7 @@ class LVCodeViewReader final : public LVBinaryReader {
Input(&Pdb), ExePath(ExePath), LogicalVisitor(this, W, Input) {}
LVCodeViewReader(const LVCodeViewReader &) = delete;
LVCodeViewReader &operator=(const LVCodeViewReader &) = delete;
~LVCodeViewReader() = default;
~LVCodeViewReader() override = default;

void getLinkageName(const llvm::object::coff_section *CoffSection,
uint32_t RelocOffset, uint32_t Offset,
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -123,7 +123,7 @@ class LVDWARFReader final : public LVBinaryReader {
Obj(Obj) {}
LVDWARFReader(const LVDWARFReader &) = delete;
LVDWARFReader &operator=(const LVDWARFReader &) = delete;
~LVDWARFReader() = default;
~LVDWARFReader() override = default;

LVAddress getCUBaseAddress() const { return CUBaseAddress; }
void setCUBaseAddress(LVAddress Address) { CUBaseAddress = Address; }
Expand Down
2 changes: 1 addition & 1 deletion llvm/include/llvm/DebugInfo/PDB/PDBSymbolTypeBuiltin.h
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ namespace pdb {
class LLVM_ABI PDBSymbolTypeBuiltin : public PDBSymbol {
DECLARE_PDB_SYMBOL_CONCRETE_TYPE(PDB_SymType::BuiltinType)
public:
~PDBSymbolTypeBuiltin();
~PDBSymbolTypeBuiltin() override;
void dump(PDBSymDumper &Dumper) const override;

FORWARD_SYMBOL_METHOD(getBuiltinType)
Expand Down
4 changes: 2 additions & 2 deletions llvm/unittests/DebugInfo/LogicalView/CompareElementsTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -75,8 +75,8 @@ class ReaderTestCompare : public LVReader {
setInstance(this);
}

Error createScopes() { return LVReader::createScopes(); }
Error printScopes() { return LVReader::printScopes(); }
Error createScopes() override { return LVReader::createScopes(); }
Error printScopes() override { return LVReader::printScopes(); }

void createElements();
void addElements(bool IsReference, bool IsTarget);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -34,7 +34,7 @@ class ReaderTest : public LVReader {
public:
ReaderTest(ScopedPrinter &W) : LVReader("", "", W) { setInstance(this); }

Error createScopes() { return LVReader::createScopes(); }
Error createScopes() override { return LVReader::createScopes(); }
};

// Helper function to add a logical element to a given scope.
Expand Down
4 changes: 2 additions & 2 deletions llvm/unittests/DebugInfo/LogicalView/LogicalElementsTest.cpp
Original file line number Diff line number Diff line change
Expand Up @@ -72,8 +72,8 @@ class ReaderTestElements : public LVReader {
setInstance(this);
}

Error createScopes() { return LVReader::createScopes(); }
Error printScopes() { return LVReader::printScopes(); }
Error createScopes() override { return LVReader::createScopes(); }
Error printScopes() override { return LVReader::printScopes(); }

void createElements();
void addElements();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ class ReaderTestSelection : public LVReader {
setInstance(this);
}

Error createScopes() { return LVReader::createScopes(); }
Error createScopes() override { return LVReader::createScopes(); }

void createElements();
void addElements();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -117,7 +117,7 @@ class ReaderTestWarningInternal : public LVReader {
setInstance(this);
}

Error createScopes() { return LVReader::createScopes(); }
Error createScopes() override { return LVReader::createScopes(); }

void setMapping();
void createElements();
Expand Down