[PATCH] D20286: [Coverage] Fix an issue where improper coverage mapping data could be loaded for an inline function.

[Igor Kudrin via llvm-commits]

ikudrin added inline comments.

================
Comment at: lib/ProfileData/Coverage/CoverageMappingReader.cpp:408
@@ -401,1 +407,3 @@
+      if (!InsertResult.second &&
+          (!FuncHash || Records[InsertResult.first->second].FunctionHash)) {
         CFR++;
----------------
vsk wrote:
> vsk wrote:
> > I'd find this a bit easier to read if InsertResult.first->second had its own variable.
> If `!InsertResult.second && FuncHash`, wdyt of creating an error if `Records[InsertResult.first->second].FunctionHash != FuncHash`?
It makes sense, but I think it would be better to make that change in another patch.


http://reviews.llvm.org/D20286