summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: 025c1ce)
raw | patch | inline | side by side (parent: 025c1ce)
author | Eric Christopher <echristo@gmail.com> | |
Thu, 6 Mar 2014 00:38:32 +0000 (00:38 +0000) | ||
committer | Eric Christopher <echristo@gmail.com> | |
Thu, 6 Mar 2014 00:38:32 +0000 (00:38 +0000) |
pointed to by the attribute, rather than the form as a first
step to determining how to hash the values. No functional change
intended.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@203044 91177308-0d34-0410-b5e6-96231b3b80d8
step to determining how to hash the values. No functional change
intended.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@203044 91177308-0d34-0410-b5e6-96231b3b80d8
lib/CodeGen/AsmPrinter/DIEHash.cpp | patch | blob | history |
index dbeb5890b00b0b0900850e182e57999ce182ac85..bc8c6957b06cd8d2de1bc0a9f69ba66592468552 100644 (file)
// reproducibility of the signature, the set of forms used in the signature
// computation is limited to the following: DW_FORM_sdata, DW_FORM_flag,
// DW_FORM_string, and DW_FORM_block.
- switch (Desc->getForm()) {
- case dwarf::DW_FORM_string:
- llvm_unreachable(
- "Add support for DW_FORM_string if we ever start emitting them again");
- case dwarf::DW_FORM_GNU_str_index:
- case dwarf::DW_FORM_strp:
- addULEB128('A');
- addULEB128(Attribute);
- addULEB128(dwarf::DW_FORM_string);
- addString(cast<DIEString>(Value)->getString());
- break;
- case dwarf::DW_FORM_data1:
- case dwarf::DW_FORM_data2:
- case dwarf::DW_FORM_data4:
- case dwarf::DW_FORM_data8:
- case dwarf::DW_FORM_udata:
- case dwarf::DW_FORM_sdata:
+
+ switch (Value->getType()) {
+ case DIEValue::isInteger: {
addULEB128('A');
addULEB128(Attribute);
- addULEB128(dwarf::DW_FORM_sdata);
- addSLEB128((int64_t)cast<DIEInteger>(Value)->getValue());
+ switch (Desc->getForm()) {
+ case dwarf::DW_FORM_data1:
+ case dwarf::DW_FORM_data2:
+ case dwarf::DW_FORM_data4:
+ case dwarf::DW_FORM_data8:
+ case dwarf::DW_FORM_udata:
+ case dwarf::DW_FORM_sdata:
+ addULEB128(dwarf::DW_FORM_sdata);
+ addSLEB128((int64_t)cast<DIEInteger>(Value)->getValue());
+ break;
+ // DW_FORM_flag_present is just flag with a value of one. We still give it a
+ // value so just use the value.
+ case dwarf::DW_FORM_flag_present:
+ case dwarf::DW_FORM_flag:
+ addULEB128(dwarf::DW_FORM_flag);
+ addULEB128((int64_t)cast<DIEInteger>(Value)->getValue());
+ break;
+ default:
+ llvm_unreachable("Unknown integer form!");
+ }
break;
- // DW_FORM_flag_present is just flag with a value of one. We still give it a
- // value so just use the value.
- case dwarf::DW_FORM_flag_present:
- case dwarf::DW_FORM_flag:
+ }
+ case DIEValue::isString:
addULEB128('A');
addULEB128(Attribute);
- addULEB128(dwarf::DW_FORM_flag);
- addULEB128((int64_t)cast<DIEInteger>(Value)->getValue());
+ addULEB128(dwarf::DW_FORM_string);
+ addString(cast<DIEString>(Value)->getString());
break;
- case dwarf::DW_FORM_exprloc:
- case dwarf::DW_FORM_block1:
- case dwarf::DW_FORM_block2:
- case dwarf::DW_FORM_block4:
- case dwarf::DW_FORM_block:
+ case DIEValue::isBlock:
+ case DIEValue::isLoc:
addULEB128('A');
addULEB128(Attribute);
addULEB128(dwarf::DW_FORM_block);
hashBlockData(cast<DIELoc>(Value)->getValues());
}
break;
- default:
- llvm_unreachable("Add support for additional forms");
+ case DIEValue::isExpr:
+ case DIEValue::isLabel:
+ case DIEValue::isDelta:
+ case DIEValue::isEntry:
+ case DIEValue::isTypeSignature:
+ case DIEValue::isLocList:
+ llvm_unreachable("Add support for additional value types.");
}
}