summaryrefslogtreecommitdiffstats
path: root/llvm/lib/IR/GCOV.cpp
diff options
context:
space:
mode:
authorDaniel Jasper <djasper@google.com>2013-12-04 08:23:33 +0000
committerDaniel Jasper <djasper@google.com>2013-12-04 08:23:33 +0000
commitc176b5d1d67b08ec05e04d17ea3de9ff74332fa5 (patch)
treedb056c47ce037134dcd1a304f25f6260af01097e /llvm/lib/IR/GCOV.cpp
parente958f9e480ddf2860495c91cd9bcafee55839703 (diff)
downloadbcm5719-llvm-c176b5d1d67b08ec05e04d17ea3de9ff74332fa5.tar.gz
bcm5719-llvm-c176b5d1d67b08ec05e04d17ea3de9ff74332fa5.zip
Revert r196358: "llvm-cov: Added support for function checksums."
This currently breaks clang/test/CodeGen/code-coverage.c. The root cause is that the newly introduced access to Funcs[j] is out of bounds. llvm-svn: 196365
Diffstat (limited to 'llvm/lib/IR/GCOV.cpp')
-rw-r--r--llvm/lib/IR/GCOV.cpp12
1 files changed, 3 insertions, 9 deletions
diff --git a/llvm/lib/IR/GCOV.cpp b/llvm/lib/IR/GCOV.cpp
index 14eb71f0300..64d53e4251a 100644
--- a/llvm/lib/IR/GCOV.cpp
+++ b/llvm/lib/IR/GCOV.cpp
@@ -125,7 +125,7 @@ bool GCOVFunction::readGCNO(GCOVBuffer &Buff, GCOV::GCOVVersion Version) {
uint32_t Dummy;
if (!Buff.readInt(Dummy)) return false; // Function header length
if (!Buff.readInt(Ident)) return false;
- if (!Buff.readInt(Checksum)) return false;
+ if (!Buff.readInt(Dummy)) return false; // Checksum #1
if (Version != GCOV::V402) {
uint32_t CfgChecksum;
if (!Buff.readInt(CfgChecksum)) return false;
@@ -212,7 +212,6 @@ bool GCOVFunction::readGCNO(GCOVBuffer &Buff, GCOV::GCOVVersion Version) {
bool GCOVFunction::readGCDA(GCOVBuffer &Buff, GCOV::GCOVVersion Version) {
uint32_t Dummy;
if (!Buff.readInt(Dummy)) return false; // Function header length
-
uint32_t GCDAIdent;
if (!Buff.readInt(GCDAIdent)) return false;
if (Ident != GCDAIdent) {
@@ -221,13 +220,8 @@ bool GCOVFunction::readGCDA(GCOVBuffer &Buff, GCOV::GCOVVersion Version) {
return false;
}
- uint32_t GCDAChecksum;
- if (!Buff.readInt(GCDAChecksum)) return false;
- if (Checksum != GCDAChecksum) {
- errs() << "Function checksums do not match: " << Checksum << " != "
- << GCDAChecksum << " (in " << Name << ").\n";
- return false;
- }
+ if (!Buff.readInt(Dummy)) return false; // Checksum #1
+
uint32_t CfgChecksum;
if (Version != GCOV::V402) {
OpenPOWER on IntegriCloud