[PATCH] D18610: [Coverage] Restore the correct count value after processing a nested region in case of combined regions.

[Igor Kudrin via llvm-commits]

ikudrin updated this revision to Diff 54664.
ikudrin added a comment.

- Reworked initialization of `I` according to Justin's comment.
- Added some explanation comments for the "Merge duplicate region" code block.

Please note that I've tried to make the behavior of the new code as close to the original as possible, apart from changes required for fix the described issue. More changes are coming, which will also change this block of code, so it's anyway not the final variant. See http://reviews.llvm.org/D18831.


http://reviews.llvm.org/D18610

Files:
  include/llvm/ProfileData/CoverageMapping.h
  lib/ProfileData/CoverageMapping.cpp
  test/tools/llvm-cov/showTemplateInstantiations.cpp
  unittests/ProfileData/CoverageMappingTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D18610.54664.patch
Type: text/x-patch
Size: 8943 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160422/72c70290/attachment.bin>