[llvm] 6148476 - [Debug-Info] change Tag type to dwarf::Tag for createAndAddDIE; NFC
Chen Zheng via llvm-commits
llvm-commits at lists.llvm.org
Thu May 13 18:15:23 PDT 2021
Author: Chen Zheng
Date: 2021-05-13T21:15:06-04:00
New Revision: 61484762e9e95aa430083ceb5bd53fe0c849728e
URL: https://github.com/llvm/llvm-project/commit/61484762e9e95aa430083ceb5bd53fe0c849728e
DIFF: https://github.com/llvm/llvm-project/commit/61484762e9e95aa430083ceb5bd53fe0c849728e.diff
LOG: [Debug-Info] change Tag type to dwarf::Tag for createAndAddDIE; NFC
Reviewed By: dblaikie
Differential Revision: https://reviews.llvm.org/D102207
Added:
Modified:
llvm/include/llvm/IR/DebugInfoMetadata.h
llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp
llvm/lib/CodeGen/AsmPrinter/DwarfUnit.h
Removed:
################################################################################
diff --git a/llvm/include/llvm/IR/DebugInfoMetadata.h b/llvm/include/llvm/IR/DebugInfoMetadata.h
index a09fee166179..da67bd8c21c5 100644
--- a/llvm/include/llvm/IR/DebugInfoMetadata.h
+++ b/llvm/include/llvm/IR/DebugInfoMetadata.h
@@ -153,7 +153,7 @@ class DINode : public MDNode {
void setTag(unsigned Tag) { SubclassData16 = Tag; }
public:
- unsigned getTag() const { return SubclassData16; }
+ dwarf::Tag getTag() const { return (dwarf::Tag)SubclassData16; }
/// Debug info flags.
///
@@ -262,7 +262,7 @@ class GenericDINode : public DINode {
/// Return a (temporary) clone of this.
TempGenericDINode clone() const { return cloneImpl(); }
- unsigned getTag() const { return SubclassData16; }
+ dwarf::Tag getTag() const { return (dwarf::Tag)SubclassData16; }
StringRef getHeader() const { return getStringOperand(0); }
MDString *getRawHeader() const { return getOperandAs<MDString>(0); }
diff --git a/llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp b/llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
index b3f1b95b16aa..cbfdd041de51 100644
--- a/llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
+++ b/llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
@@ -273,9 +273,9 @@ static StringRef getPrettyScopeName(const DIScope *Scope) {
return "<unnamed-tag>";
case dwarf::DW_TAG_namespace:
return "`anonymous namespace'";
+ default:
+ return StringRef();
}
-
- return StringRef();
}
const DISubprogram *CodeViewDebug::collectParentScopeNames(
@@ -1487,6 +1487,9 @@ static bool shouldEmitUdt(const DIType *T) {
case dwarf::DW_TAG_class_type:
case dwarf::DW_TAG_union_type:
return false;
+ default:
+ // do nothing.
+ ;
}
}
}
@@ -2032,10 +2035,13 @@ static MethodKind translateMethodKindFlags(const DISubprogram *SP,
static TypeRecordKind getRecordKind(const DICompositeType *Ty) {
switch (Ty->getTag()) {
- case dwarf::DW_TAG_class_type: return TypeRecordKind::Class;
- case dwarf::DW_TAG_structure_type: return TypeRecordKind::Struct;
+ case dwarf::DW_TAG_class_type:
+ return TypeRecordKind::Class;
+ case dwarf::DW_TAG_structure_type:
+ return TypeRecordKind::Struct;
+ default:
+ llvm_unreachable("unexpected tag");
}
- llvm_unreachable("unexpected tag");
}
/// Return ClassOptions that should be present on both the forward declaration
diff --git a/llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp b/llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp
index 2b171ae3b6e8..a28dbb3211c9 100644
--- a/llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp
+++ b/llvm/lib/CodeGen/AsmPrinter/DwarfUnit.cpp
@@ -385,8 +385,8 @@ void DwarfUnit::addDIEEntry(DIE &Die, dwarf::Attribute Attribute,
Entry);
}
-DIE &DwarfUnit::createAndAddDIE(unsigned Tag, DIE &Parent, const DINode *N) {
- DIE &Die = Parent.addChild(DIE::get(DIEValueAllocator, (dwarf::Tag)Tag));
+DIE &DwarfUnit::createAndAddDIE(dwarf::Tag Tag, DIE &Parent, const DINode *N) {
+ DIE &Die = Parent.addChild(DIE::get(DIEValueAllocator, Tag));
if (N)
insertDIE(N, &Die);
return Die;
diff --git a/llvm/lib/CodeGen/AsmPrinter/DwarfUnit.h b/llvm/lib/CodeGen/AsmPrinter/DwarfUnit.h
index 75bc3de14d61..df1ead03dc05 100644
--- a/llvm/lib/CodeGen/AsmPrinter/DwarfUnit.h
+++ b/llvm/lib/CodeGen/AsmPrinter/DwarfUnit.h
@@ -257,7 +257,7 @@ class DwarfUnit : public DIEUnit {
/// Create a DIE with the given Tag, add the DIE to its parent, and
/// call insertDIE if MD is not null.
- DIE &createAndAddDIE(unsigned Tag, DIE &Parent, const DINode *N = nullptr);
+ DIE &createAndAddDIE(dwarf::Tag Tag, DIE &Parent, const DINode *N = nullptr);
bool useSegmentedStringOffsetsTable() const {
return DD->useSegmentedStringOffsetsTable();
More information about the llvm-commits
mailing list