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

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

Issue 2448833004: [wasm] Fix binary search for asm.js offsets (Closed)
Patch Set: Created 4 years, 2 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
« no previous file with comments | « no previous file | test/mjsunit/wasm/asm-wasm-stack.js » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/wasm/wasm-debug.cc
diff --git a/src/wasm/wasm-debug.cc b/src/wasm/wasm-debug.cc
index a5ac40a8948080fc71cc85e6a404f0f3ac367d9c..aa1357a2847504dc041ce70ccf43a01e655a74a4 100644
--- a/src/wasm/wasm-debug.cc
+++ b/src/wasm/wasm-debug.cc
@@ -303,7 +303,7 @@ int WasmDebugInfo::GetAsmJsSourcePosition(Handle<WasmDebugInfo> debug_info,
DCHECK_LT(left, right);
while (right - left > 1) {
int mid = left + (right - left) / 2;
- if (offset_table->get_int(2 * mid) < byte_offset) {
+ if (offset_table->get_int(2 * mid) <= byte_offset) {
left = mid;
} else {
right = mid;
« no previous file with comments | « no previous file | test/mjsunit/wasm/asm-wasm-stack.js » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698