[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