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

David Li via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 14 09:53:31 PDT 2016


davidxl added inline comments.

================
Comment at: llvm/trunk/lib/ProfileData/CoverageMapping.cpp:403
@@ +402,3 @@
+  for (++I; I != Last; ++I) {
+    if (Active->startLoc() != I->startLoc() ||
+        Active->endLoc() != I->endLoc()) {
----------------
davidxl wrote:
> Can you just do
> 
> // Find a new region
> if (Active->StartLoc() != ... ) {
>    Active = I;
>    continue;
> }
Discard this comment -- your code is correct.


http://reviews.llvm.org/D18610





More information about the llvm-commits mailing list