AuroraMiddleware/v8
1
0
Fork 0
Code Issues 2 Pull Requests Releases Wiki Activity

[coverage] Recognize binary block mode as block coverage

Browse Source 
Bug: v8:6000
Change-Id: I2853d44d8bcf34b28630594cc9c2782046bd4b28
Reviewed-on: https://chromium-review.googlesource.com/573900
Reviewed-by: Yang Guo <yangguo@chromium.org>
Commit-Queue: Jakob Gruber <jgruber@chromium.org>
Cr-Commit-Position: refs/heads/master@{#46700}
This commit is contained in:
jgruber 2017-07-17 10:51:46 +02:00 committed by Commit Bot
parent 790c922133
commit b34023e876
2 changed files with 32 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
src/debug/debug-coverage.cc
View File

@ -325,6 +325,7 @@ void ResetAllBlockCounts(SharedFunctionInfo* shared) {
bool IsBlockMode(debug::Coverage::Mode mode) {
switch (mode) {
case debug::Coverage::kBlockBinary:
case debug::Coverage::kBlockCount:
return true;
default:

37
test/inspector/cpu-profiler/coverage-block-expected.txt
View File

@ -825,7 +825,7 @@ Running test: testPreciseBinaryCoverage
functions : [
[0] : {
functionName :
isBlockCoverage : false
isBlockCoverage : true
ranges : [
[0] : {
count : 1
@ -836,13 +836,28 @@ Running test: testPreciseBinaryCoverage
}
[1] : {
functionName : fib
isBlockCoverage : false
isBlockCoverage : true
ranges : [
[0] : {
count : 1
endOffset : 73
startOffset : 1
}
[1] : {
count : 1
endOffset : 41
startOffset : 32
}
[2] : {
count : 1
endOffset : 71
startOffset : 41
}
[3] : {
count : 0
endOffset : 73
startOffset : 71
}
]
}
[2] : {
@ -858,13 +873,18 @@ Running test: testPreciseBinaryCoverage
}
[3] : {
functionName : iife
isBlockCoverage : false
isBlockCoverage : true
ranges : [
[0] : {
count : 1
endOffset : 208
startOffset : 177
}
[1] : {
count : 0
endOffset : 208
startOffset : 206
}
]
}
]
@ -910,13 +930,18 @@ Running test: testPreciseBinaryCoverage
functions : [
[0] : {
functionName : is_optimized
isBlockCoverage : false
isBlockCoverage : true
ranges : [
[0] : {
count : 1
endOffset : 175
startOffset : 74
}
[1] : {
count : 0
endOffset : 175
startOffset : 173
}
]
}
]
@ -927,7 +952,7 @@ Running test: testPreciseBinaryCoverage
functions : [
[0] : {
functionName :
isBlockCoverage : false
isBlockCoverage : true
ranges : [
[0] : {
count : 1
@ -944,7 +969,7 @@ Running test: testPreciseBinaryCoverage
functions : [
[0] : {
functionName :
isBlockCoverage : false
isBlockCoverage : true
ranges : [
[0] : {
count : 1