maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

DIEHash: Do not use shallow type hashing for unnamed types 

llvm-svn: 193361
This commit is contained in:
David Blaikie 2013-10-24 17:53:58 +00:00
parent afcb9656c3
commit 32744412d2
2 changed files with 41 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
llvm/lib/CodeGen/AsmPrinter/DIEHash.cpp
View File

@ -242,11 +242,13 @@ void DIEHash::hashDIEEntry(dwarf::Attribute Attribute, dwarf::Tag Tag,
// there's a decl/def difference in the containing type of a // there's a decl/def difference in the containing type of a
// ptr_to_member_type, but it's what DWARF says, for some reason. // ptr_to_member_type, but it's what DWARF says, for some reason.
Attribute == dwarf::DW_AT_type) { Attribute == dwarf::DW_AT_type) {
// [FIXME] ... has a DW_AT_name attribute, // ... has a DW_AT_name attribute,
hashShallowTypeReference(Attribute, Entry, StringRef Name = getDIEStringAttr(Entry, dwarf::DW_AT_name);
getDIEStringAttr(Entry, dwarf::DW_AT_name)); if (!Name.empty()) {
hashShallowTypeReference(Attribute, Entry, Name);
return; return;
} }
}
unsigned &DieNumber = Numbering[&Entry]; unsigned &DieNumber = Numbering[&Entry];
if (DieNumber) { if (DieNumber) {

35
llvm/unittests/CodeGen/DIEHashTest.cpp
View File

@ -442,4 +442,39 @@ TEST(DIEHashTest, PtrToMemberDeclDefMisMatch) {
// and a definition in another. // and a definition in another.
ASSERT_NE(MD5ResDef, MD5ResDecl); ASSERT_NE(MD5ResDef, MD5ResDecl);
} }
// struct { } a;
// struct foo { decltype(a) mem; };
TEST(DIEHashTest, RefUnnamedType) {
DIEInteger Zero(0);
DIEInteger One(1);
DIEInteger Eight(8);
DIEString FooStr(&Zero, "foo");
DIEString MemStr(&Zero, "mem");
DIE Unnamed(dwarf::DW_TAG_structure_type);
Unnamed.addValue(dwarf::DW_AT_byte_size, dwarf::DW_FORM_data1, &One);
DIE Foo(dwarf::DW_TAG_structure_type);
Foo.addValue(dwarf::DW_AT_byte_size, dwarf::DW_FORM_data1, &Eight);
Foo.addValue(dwarf::DW_AT_name, dwarf::DW_FORM_strp, &FooStr);
DIE *Mem = new DIE(dwarf::DW_TAG_member);
Mem->addValue(dwarf::DW_AT_name, dwarf::DW_FORM_strp, &MemStr);
Mem->addValue(dwarf::DW_AT_data_member_location, dwarf::DW_FORM_data1, &Zero);
DIE UnnamedPtr(dwarf::DW_TAG_pointer_type);
UnnamedPtr.addValue(dwarf::DW_AT_byte_size, dwarf::DW_FORM_data1, &Eight);
DIEEntry UnnamedRef(&Unnamed);
UnnamedPtr.addValue(dwarf::DW_AT_type, dwarf::DW_FORM_ref4, &UnnamedRef);
DIEEntry UnnamedPtrRef(&UnnamedPtr);
Mem->addValue(dwarf::DW_AT_type, dwarf::DW_FORM_ref4, &UnnamedPtrRef);
Foo.addChild(Mem);
uint64_t MD5Res = DIEHash().computeTypeSignature(&Foo);
ASSERT_EQ(0x954e026f01c02529ULL, MD5Res);
}
} }