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

Issue 2105303002: [wasm] Disassemble wasm code from script (Closed)

Created:
4 years, 5 months ago by titzer
Modified:
4 years, 5 months ago
Reviewers:
titzer, ahaas, Yang
CC:
v8-reviews_googlegroups.com
Base URL:
https://chromium.googlesource.com/v8/v8.git@master
Target Ref:
refs/pending/heads/master
Project:
v8
Visibility:
Public.

Description

[wasm] Disassemble wasm code from script This stores the wasm object and the function index in the script, and adds functions to get the disassembled wasm code as well as the offset table mapping from byte position to line and column in the disassembly solely from the script. This will be used to show "ui source code" in DevTools, and map raw locations from the stack trace into this code view. R=yangguo@chromium.org, ahaas@chromium.org, titzer@chromium.org BUG=chromium:613110 patch from issue 2063013004 at patchset 80001 (http://crrev.com/2063013004#ps80001) Committed: https://crrev.com/d249efd705cd851e8441ee3cc804669812ba1694 Cr-Commit-Position: refs/heads/master@{#37430}

Patch Set 1 #

Total comments: 5

Patch Set 2 : [wasm] Disassemble wasm code from script #

Patch Set 3 : [wasm] Disassemble wasm code from script #

Unified diffs Side-by-side diffs Delta from patch set Stats (+257 lines, -43 lines) Patch
M src/debug/debug.js View 1 2 6 chunks +32 lines, -20 lines 0 comments Download
M src/objects.h View 1 chunk +9 lines, -1 line 0 comments Download
M src/objects-inl.h View 2 chunks +31 lines, -15 lines 0 comments Download
M src/runtime/runtime.h View 1 chunk +3 lines, -1 line 0 comments Download
M src/runtime/runtime-debug.cc View 2 chunks +31 lines, -0 lines 0 comments Download
M src/wasm/ast-decoder.cc View 1 chunk +1 line, -0 lines 0 comments Download
M src/wasm/wasm-debug.cc View 3 chunks +11 lines, -6 lines 0 comments Download
A test/mjsunit/wasm/debug-disassembly.js View 1 chunk +139 lines, -0 lines 0 comments Download

Messages

Total messages: 15 (5 generated)
titzer
4 years, 5 months ago (2016-06-29 12:15:43 UTC) #1
titzer
On 2016/06/29 12:15:43, titzer wrote: PTAL
4 years, 5 months ago (2016-06-29 12:47:56 UTC) #2
ahaas
lgtm https://codereview.chromium.org/2105303002/diff/1/src/debug/debug.js File src/debug/debug.js (right): https://codereview.chromium.org/2105303002/diff/1/src/debug/debug.js#newcode866 src/debug/debug.js:866: return null; nit: wouldn't it be more javascripty ...
4 years, 5 months ago (2016-06-29 14:27:27 UTC) #3
titzer
https://codereview.chromium.org/2105303002/diff/1/src/debug/debug.js File src/debug/debug.js (right): https://codereview.chromium.org/2105303002/diff/1/src/debug/debug.js#newcode866 src/debug/debug.js:866: return null; On 2016/06/29 14:27:27, ahaas wrote: > nit: ...
4 years, 5 months ago (2016-06-30 08:12:51 UTC) #4
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2105303002/40001
4 years, 5 months ago (2016-06-30 09:25:42 UTC) #7
commit-bot: I haz the power
Try jobs failed on following builders: v8_presubmit on master.tryserver.v8 (JOB_FAILED, http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/18519)
4 years, 5 months ago (2016-06-30 09:29:51 UTC) #9
Yang
On 2016/06/30 09:29:51, commit-bot: I haz the power wrote: > Try jobs failed on following ...
4 years, 5 months ago (2016-06-30 09:50:41 UTC) #10
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/v2/patch-status/codereview.chromium.org/2105303002/40001
4 years, 5 months ago (2016-06-30 09:52:43 UTC) #12
commit-bot: I haz the power
Committed patchset #3 (id:40001)
4 years, 5 months ago (2016-06-30 09:54:46 UTC) #13
commit-bot: I haz the power
4 years, 5 months ago (2016-06-30 09:57:18 UTC) #15
Message was sent while issue was closed.
Patchset 3 (id:??) landed as
https://crrev.com/d249efd705cd851e8441ee3cc804669812ba1694
Cr-Commit-Position: refs/heads/master@{#37430}

Powered by Google App Engine
This is Rietveld 408576698