summaryrefslogtreecommitdiff
path: root/lib/DebugInfo
diff options
context:
space:
mode:
authorZachary Turner <zturner@google.com>2017-11-28 23:57:13 +0000
committerZachary Turner <zturner@google.com>2017-11-28 23:57:13 +0000
commitdc4f5ff552f66fb72a5d7916f88afb8834f813dc (patch)
treea8808f0124817878e473b3bfff74303cac79baaa /lib/DebugInfo
parent2677b3dc56af81860c9a9774e5e7733afb7c504b (diff)
[NFC] Minor cleanups in CodeView TypeTableBuilder.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@319260 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/DebugInfo')
-rw-r--r--lib/DebugInfo/CodeView/TypeTableBuilder.cpp17
1 files changed, 7 insertions, 10 deletions
diff --git a/lib/DebugInfo/CodeView/TypeTableBuilder.cpp b/lib/DebugInfo/CodeView/TypeTableBuilder.cpp
index bce636f3894..7b02b175bd6 100644
--- a/lib/DebugInfo/CodeView/TypeTableBuilder.cpp
+++ b/lib/DebugInfo/CodeView/TypeTableBuilder.cpp
@@ -31,9 +31,8 @@ using namespace llvm::codeview;
namespace {
struct HashedType {
- uint64_t Hash;
- const uint8_t *Data;
- unsigned Size; // FIXME: Go to uint16_t?
+ unsigned Hash;
+ ArrayRef<uint8_t> Data;
TypeIndex Index;
};
@@ -67,9 +66,9 @@ template <> struct DenseMapInfo<HashedTypePtr> {
HashedType *RHS = RHSP.Ptr;
if (RHS == getEmptyKey().Ptr || RHS == getTombstoneKey().Ptr)
return LHS == RHS;
- if (LHS->Hash != RHS->Hash || LHS->Size != RHS->Size)
+ if (LHS->Hash != RHS->Hash)
return false;
- return ::memcmp(LHS->Data, RHS->Data, LHS->Size) == 0;
+ return LHS->Data == RHS->Data;
}
};
@@ -111,8 +110,7 @@ TypeIndex TypeHasher::getOrCreateRecord(ArrayRef<uint8_t> &Record,
assert(Record.size() % 4 == 0 && "Record is not aligned to 4 bytes!");
// Compute the hash up front so we can store it in the key.
- HashedType TempHashedType = {hash_value(Record), Record.data(),
- unsigned(Record.size()), TI};
+ HashedType TempHashedType = {hash_value(Record), Record, TI};
auto Result = HashedRecords.insert(HashedTypePtr(&TempHashedType));
HashedType *&Hashed = Result.first->Ptr;
@@ -124,12 +122,11 @@ TypeIndex TypeHasher::getOrCreateRecord(ArrayRef<uint8_t> &Record,
uint8_t *Stable = RecordStorage.Allocate<uint8_t>(Record.size());
memcpy(Stable, Record.data(), Record.size());
- Hashed->Data = Stable;
- assert(Hashed->Size == Record.size());
+ Hashed->Data = makeArrayRef(Stable, Record.size());
}
// Update the caller's copy of Record to point a stable copy.
- Record = ArrayRef<uint8_t>(Hashed->Data, Hashed->Size);
+ Record = Hashed->Data;
return Hashed->Index;
}