Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(156)

Unified Diff: src/debug/debug-coverage.cc

Issue 2882973002: [coverage] Block coverage with support for IfStatements (Closed)
Patch Set: Comment nit Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: src/debug/debug-coverage.cc
diff --git a/src/debug/debug-coverage.cc b/src/debug/debug-coverage.cc
index 76f7b3fd5ac0ef155fffcf0213f4d0a5c68bf656..973ac489196d37d113e75c5365d7ca5eb24a52bc 100644
--- a/src/debug/debug-coverage.cc
+++ b/src/debug/debug-coverage.cc
@@ -57,6 +57,40 @@ bool CompareSharedFunctionInfo(SharedFunctionInfo* a, SharedFunctionInfo* b) {
if (a_start == b_start) return a->end_position() > b->end_position();
return a_start < b_start;
}
+
+bool CompareCoverageBlock(const CoverageBlock& a, const CoverageBlock& b) {
+ DCHECK(a.start != kNoSourcePosition && a.end != kNoSourcePosition);
+ DCHECK(b.start != kNoSourcePosition && b.end != kNoSourcePosition);
+ if (a.start == b.start) return a.end > b.end;
+ return a.start < b.start;
+}
+
+std::vector<CoverageBlock> GetSortedBlockData(Isolate* isolate,
+ SharedFunctionInfo* shared) {
+ DCHECK(FLAG_block_coverage);
+
+ std::vector<CoverageBlock> result;
+ Object* maybe_coverage_info = shared->coverage_info();
+
+ if (!maybe_coverage_info->IsCoverageInfo()) return result;
+ CoverageInfo* coverage_info = CoverageInfo::cast(maybe_coverage_info);
+
+ for (int i = 0; i < coverage_info->SlotCount(); i++) {
+ const int start_pos = coverage_info->FromSourcePosition(i);
+ const int until_pos = coverage_info->ToSourcePosition(i);
+ const int count = coverage_info->BlockCount(i);
+
+ DCHECK(start_pos != kNoSourcePosition);
+ DCHECK(until_pos != kNoSourcePosition);
+
+ result.emplace_back(start_pos, until_pos, count);
+ }
+
+ // Sort according to the block nesting structure.
+ std::sort(result.begin(), result.end(), CompareCoverageBlock);
+
+ return result;
+}
} // anonymous namespace
Coverage* Coverage::CollectPrecise(Isolate* isolate) {
@@ -79,6 +113,7 @@ Coverage* Coverage::Collect(Isolate* isolate,
SharedToCounterMap counter_map;
switch (isolate->code_coverage_mode()) {
+ case v8::debug::Coverage::kBlockBinary:
case v8::debug::Coverage::kPreciseBinary:
case v8::debug::Coverage::kPreciseCount: {
bool reset_count = collectionMode != v8::debug::Coverage::kBestEffort;
@@ -151,6 +186,7 @@ Coverage* Coverage::Collect(Isolate* isolate,
switch (collectionMode) {
case v8::debug::Coverage::kPreciseCount:
break;
+ case v8::debug::Coverage::kBlockBinary:
case v8::debug::Coverage::kPreciseBinary:
count = info->has_reported_binary_coverage() ? 0 : 1;
info->set_has_reported_binary_coverage(true);
@@ -167,6 +203,11 @@ Coverage* Coverage::Collect(Isolate* isolate,
Handle<String> name(info->DebugName(), isolate);
nesting.push_back(functions->size());
functions->emplace_back(start, end, count, name);
+
+ if (FLAG_block_coverage) {
+ CoverageFunction* function = &functions->back();
+ function->blocks = GetSortedBlockData(isolate, info);
+ }
}
}
@@ -181,6 +222,7 @@ void Coverage::SelectMode(Isolate* isolate, debug::Coverage::Mode mode) {
case debug::Coverage::kBestEffort:
isolate->SetCodeCoverageList(isolate->heap()->undefined_value());
break;
+ case debug::Coverage::kBlockBinary:
case debug::Coverage::kPreciseBinary:
case debug::Coverage::kPreciseCount: {
HandleScope scope(isolate);

Powered by Google App Engine
This is Rietveld 408576698