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

Issue 2087453002: PPC/s390: [wasm] Separate compilation from instantiation (Closed)

Created:
4 years, 6 months ago by JaideepBajwa
Modified:
4 years, 6 months ago
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

PPC/s390: [wasm] Separate compilation from instantiation Port c1d01aea117c62759cc2be1805493ed59218558c Original commit message: Compilation of wasm functions happens before instantiation. Imports are linked afterwards, at instantiation time. Globals and memory are also allocated and then tied in via relocation at instantiation time. This paves the way for implementing Wasm.compile, a prerequisite to offering the compiled code serialization feature. Currently, the WasmModule::Compile method just returns a fixed array containing the code objects. More appropriate modeling of the compiled module to come. Opportunistically centralized the logic on how to update memory references, size, and globals, since that logic is the exact same on each architecture, except for the actual storing of values back in the instruction stream. R=mtrofin@chromium.org, joransiu@ca.ibm.com, jyan@ca.ibm.com, michael_dawson@ca.ibm.com, mbrandy@us.ibm.com BUG=v8:5072 LOG=N Committed: https://crrev.com/8071e21ce5ce133d91ab749cd7f9bde189cdddb6 Cr-Commit-Position: refs/heads/master@{#37116}

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+14 lines, -68 lines) Patch
M src/ppc/assembler-ppc.cc View 1 chunk +7 lines, -34 lines 0 comments Download
M src/s390/assembler-s390.cc View 1 chunk +7 lines, -34 lines 0 comments Download

Messages

Total messages: 8 (2 generated)
JaideepBajwa
PTAL
4 years, 6 months ago (2016-06-20 15:41:07 UTC) #1
Mircea Trofin
On 2016/06/20 15:41:07, JaideepBajwa wrote: > PTAL lgtm
4 years, 6 months ago (2016-06-20 15:42:27 UTC) #2
john.yan
lgtm
4 years, 6 months ago (2016-06-20 18:46:15 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/2087453002/1
4 years, 6 months ago (2016-06-20 18:46:51 UTC) #5
commit-bot: I haz the power
Committed patchset #1 (id:1)
4 years, 6 months ago (2016-06-20 19:09:28 UTC) #6
commit-bot: I haz the power
4 years, 6 months ago (2016-06-20 19:12:17 UTC) #8
Message was sent while issue was closed.
Patchset 1 (id:??) landed as
https://crrev.com/8071e21ce5ce133d91ab749cd7f9bde189cdddb6
Cr-Commit-Position: refs/heads/master@{#37116}

Powered by Google App Engine
This is Rietveld 408576698