[llvm] r274632 - Revert "Emit CodeView type records for nested classes."
Adrian McCarthy via llvm-commits
llvm-commits at lists.llvm.org
Wed Jul 6 08:14:10 PDT 2016
Author: amccarth
Date: Wed Jul 6 10:14:10 2016
New Revision: 274632
URL: http://llvm.org/viewvc/llvm-project?rev=274632&view=rev
Log:
Revert "Emit CodeView type records for nested classes."
This reverts commit 256b29322c827a2d94da56468c936596f5509032.
Removed:
llvm/trunk/test/DebugInfo/COFF/types-nested-class.ll
Modified:
llvm/trunk/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
llvm/trunk/lib/CodeGen/AsmPrinter/CodeViewDebug.h
Modified: llvm/trunk/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp?rev=274632&r1=274631&r2=274632&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp (original)
+++ llvm/trunk/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp Wed Jul 6 10:14:10 2016
@@ -1414,8 +1414,6 @@ struct llvm::ClassInfo {
MemberList Members;
// Direct overloaded methods gathered by name.
MethodsMap Methods;
-
- std::vector<const DICompositeType *> NestedClasses;
};
void CodeViewDebug::clear() {
@@ -1468,8 +1466,8 @@ ClassInfo CodeViewDebug::collectClassInf
// friends in the past, but modern versions do not.
}
// FIXME: Get Clang to emit function virtual table here and handle it.
- } else if (auto *Composite = dyn_cast<DICompositeType>(Element)) {
- Info.NestedClasses.push_back(Composite);
+ // FIXME: Get clang to emit nested types here and do something with
+ // them.
}
// Skip other unrecognized kinds of elements.
}
@@ -1498,12 +1496,7 @@ TypeIndex CodeViewDebug::lowerCompleteTy
TypeIndex FieldTI;
TypeIndex VShapeTI;
unsigned FieldCount;
- bool ContainsNestedClass;
- std::tie(FieldTI, VShapeTI, FieldCount, ContainsNestedClass) =
- lowerRecordFieldList(Ty);
-
- if (ContainsNestedClass)
- CO |= ClassOptions::ContainsNestedClass;
+ std::tie(FieldTI, VShapeTI, FieldCount) = lowerRecordFieldList(Ty);
std::string FullName = getFullyQualifiedName(Ty);
@@ -1539,13 +1532,7 @@ TypeIndex CodeViewDebug::lowerCompleteTy
ClassOptions CO = getCommonClassOptions(Ty);
TypeIndex FieldTI;
unsigned FieldCount;
- bool ContainsNestedClass;
- std::tie(FieldTI, std::ignore, FieldCount, ContainsNestedClass) =
- lowerRecordFieldList(Ty);
-
- if (ContainsNestedClass)
- CO |= ClassOptions::ContainsNestedClass;
-
+ std::tie(FieldTI, std::ignore, FieldCount) = lowerRecordFieldList(Ty);
uint64_t SizeInBytes = Ty->getSizeInBits() / 8;
std::string FullName = getFullyQualifiedName(Ty);
@@ -1563,7 +1550,7 @@ TypeIndex CodeViewDebug::lowerCompleteTy
return UnionTI;
}
-std::tuple<TypeIndex, TypeIndex, unsigned, bool>
+std::tuple<TypeIndex, TypeIndex, unsigned>
CodeViewDebug::lowerRecordFieldList(const DICompositeType *Ty) {
// Manually count members. MSVC appears to count everything that generates a
// field list record. Each individual overload in a method overload group
@@ -1658,17 +1645,8 @@ CodeViewDebug::lowerRecordFieldList(cons
OverloadedMethodRecord(Methods.size(), MethodList, Name));
}
}
-
- // Create nested classes.
- for (const DICompositeType *Nested : Info.NestedClasses) {
- NestedTypeRecord R(getTypeIndex(DITypeRef(Nested)), Nested->getName());
- Fields.writeNestedType(R);
- MemberCount++;
- }
-
TypeIndex FieldTI = TypeTable.writeFieldList(Fields);
- return std::make_tuple(FieldTI, TypeIndex(), MemberCount,
- !Info.NestedClasses.empty());
+ return std::make_tuple(FieldTI, TypeIndex(), MemberCount);
}
TypeIndex CodeViewDebug::getVBPTypeIndex() {
Modified: llvm/trunk/lib/CodeGen/AsmPrinter/CodeViewDebug.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/CodeGen/AsmPrinter/CodeViewDebug.h?rev=274632&r1=274631&r2=274632&view=diff
==============================================================================
--- llvm/trunk/lib/CodeGen/AsmPrinter/CodeViewDebug.h (original)
+++ llvm/trunk/lib/CodeGen/AsmPrinter/CodeViewDebug.h Wed Jul 6 10:14:10 2016
@@ -275,7 +275,7 @@ class LLVM_LIBRARY_VISIBILITY CodeViewDe
/// Common record member lowering functionality for record types, which are
/// structs, classes, and unions. Returns the field list index and the member
/// count.
- std::tuple<codeview::TypeIndex, codeview::TypeIndex, unsigned, bool>
+ std::tuple<codeview::TypeIndex, codeview::TypeIndex, unsigned>
lowerRecordFieldList(const DICompositeType *Ty);
/// Inserts {{Node, ClassTy}, TI} into TypeIndices and checks for duplicates.
Removed: llvm/trunk/test/DebugInfo/COFF/types-nested-class.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/COFF/types-nested-class.ll?rev=274631&view=auto
==============================================================================
--- llvm/trunk/test/DebugInfo/COFF/types-nested-class.ll (original)
+++ llvm/trunk/test/DebugInfo/COFF/types-nested-class.ll (removed)
@@ -1,98 +0,0 @@
-; RUN: llc < %s -filetype=obj | llvm-readobj - -codeview | FileCheck %s
-
-; C++ source to regenerate:
-; $ cat hello.cpp
-; struct A {
-; struct Nested {};
-; } a;
-; $ clang hello.cpp -S -emit-llvm -g -gcodeview -o t.ll
-
-; CHECK: CodeViewTypes [
-; CHECK: Section: .debug$T (5)
-; CHECK: Magic: 0x4
-; CHECK: Struct (0x1000) {
-; CHECK: TypeLeafKind: LF_STRUCTURE (0x1505)
-; CHECK: MemberCount: 0
-; CHECK: Properties [ (0x280)
-; CHECK: ForwardReference (0x80)
-; CHECK: HasUniqueName (0x200)
-; CHECK: ]
-; CHECK: FieldList: 0x0
-; CHECK: DerivedFrom: 0x0
-; CHECK: VShape: 0x0
-; CHECK: SizeOf: 0
-; CHECK: Name: A
-; CHECK: LinkageName: .?AUA@@
-; CHECK: }
-; CHECK: Struct (0x1001) {
-; CHECK: TypeLeafKind: LF_STRUCTURE (0x1505)
-; CHECK: MemberCount: 0
-; CHECK: Properties [ (0x280)
-; CHECK: ForwardReference (0x80)
-; CHECK: HasUniqueName (0x200)
-; CHECK: ]
-; CHECK: FieldList: 0x0
-; CHECK: DerivedFrom: 0x0
-; CHECK: VShape: 0x0
-; CHECK: SizeOf: 0
-; CHECK: Name: A::Nested
-; CHECK: LinkageName: .?AUNested at A@@
-; CHECK: }
-; CHECK: FieldList (0x1002) {
-; CHECK: TypeLeafKind: LF_FIELDLIST (0x1203)
-; CHECK: NestedType {
-; CHECK: Type: A::Nested (0x1001)
-; CHECK: Name: Nested
-; CHECK: }
-; CHECK: }
-; CHECK: Struct (0x1003) {
-; CHECK: TypeLeafKind: LF_STRUCTURE (0x1505)
-; CHECK: MemberCount: 1
-; CHECK: Properties [ (0x210)
-; CHECK: ContainsNestedClass (0x10)
-; CHECK: HasUniqueName (0x200)
-; CHECK: ]
-; CHECK: FieldList: <field list> (0x1002)
-; CHECK: DerivedFrom: 0x0
-; CHECK: VShape: 0x0
-; CHECK: SizeOf: 1
-; CHECK: Name: A
-; CHECK: LinkageName: .?AUA@@
-; CHECK: }
-; CHECK: StringId (0x1004) {
-; CHECK: TypeLeafKind: LF_STRING_ID (0x1605)
-; CHECK: Id: 0x0
-; CHECK: StringData: D:\src\hello\hello.cpp
-; CHECK: }
-; CHECK: UdtSourceLine (0x1005) {
-; CHECK: TypeLeafKind: LF_UDT_SRC_LINE (0x1606)
-; CHECK: UDT: A (0x1003)
-; CHECK: SourceFile: D:\src\hello\hello.cpp (0x1004)
-; CHECK: LineNumber: 1
-; CHECK: }
-; CHECK: ]
-
-; ModuleID = 'hello.cpp'
-source_filename = "hello.cpp"
-target datalayout = "e-m:x-p:32:32-i64:64-f80:32-n8:16:32-a:0:32-S32"
-target triple = "i686-pc-windows-msvc19.0.23918"
-
-%struct.A = type { i8 }
-
-@"\01?a@@3UA@@A" = global %struct.A zeroinitializer, align 1
-
-!llvm.dbg.cu = !{!0}
-!llvm.module.flags = !{!8, !9}
-!llvm.ident = !{!10}
-
-!0 = distinct !DICompileUnit(language: DW_LANG_C_plus_plus, file: !1, producer: "clang version 3.9.0 ", isOptimized: false, runtimeVersion: 0, emissionKind: FullDebug, enums: !2, globals: !3)
-!1 = !DIFile(filename: "hello.cpp", directory: "D:\5Csrc\5Chello")
-!2 = !{}
-!3 = !{!4}
-!4 = distinct !DIGlobalVariable(name: "a", linkageName: "\01?a@@3UA@@A", scope: !0, file: !1, line: 3, type: !5, isLocal: false, isDefinition: true, variable: %struct.A* @"\01?a@@3UA@@A")
-!5 = distinct !DICompositeType(tag: DW_TAG_structure_type, name: "A", file: !1, line: 1, size: 8, align: 8, elements: !6, identifier: ".?AUA@@")
-!6 = !{!7}
-!7 = !DICompositeType(tag: DW_TAG_structure_type, name: "Nested", scope: !5, file: !1, line: 2, size: 8, align: 8, flags: DIFlagFwdDecl, identifier: ".?AUNested at A@@")
-!8 = !{i32 2, !"CodeView", i32 1}
-!9 = !{i32 2, !"Debug Info Version", i32 3}
-!10 = !{!"clang version 3.9.0 "}
More information about the llvm-commits
mailing list