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

Issue 1531243003: Turn on wasm flags all the time, add a reference from wasm functions to the module. (Closed)

Created:
5 years ago by bradn
Modified:
5 years ago
Reviewers:
Jarin, ahaas, Camillo Bruni
CC:
Michael Hablich, 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

Turn on wasm flags all the time, add a reference from wasm functions to the module. Add an internal field to each wasm function to keep a reference to the module. (So the GC can do the right thing when you only hold references to wasm functions but not the module). Use Realloc carefully, to avoid copying from out of bounds. Make snprintf use platform independent. Don't disconnect external arraybuffers provided for the heap. R=ahaas@chromium.org BUG= Committed: https://crrev.com/7b775118b98eba613ab3a5936f9c489ed7d1b45a Cr-Commit-Position: refs/heads/master@{#32951}

Patch Set 1 #

Patch Set 2 : fix #

Patch Set 3 : fxi #

Patch Set 4 : add missing file #

Unified diffs Side-by-side diffs Delta from patch set Stats (+47 lines, -44 lines) Patch
M src/compiler.h View 1 chunk +2 lines, -1 line 0 comments Download
M src/compiler/wasm-compiler.h View 1 2 3 1 chunk +3 lines, -5 lines 0 comments Download
M src/compiler/wasm-compiler.cc View 1 2 12 chunks +31 lines, -29 lines 0 comments Download
M src/contexts.h View 1 chunk +1 line, -0 lines 0 comments Download
M src/factory.cc View 1 chunk +2 lines, -0 lines 0 comments Download
M src/flag-definitions.h View 1 chunk +1 line, -3 lines 0 comments Download
M src/wasm/wasm-js.cc View 1 4 chunks +7 lines, -6 lines 0 comments Download

Messages

Total messages: 26 (13 generated)
bradn
5 years ago (2015-12-17 12:46:07 UTC) #2
ahaas
lgtm
5 years ago (2015-12-17 12:58:39 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1531243003/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1531243003/40001
5 years ago (2015-12-17 13:01:05 UTC) #5
commit-bot: I haz the power
Try jobs failed on following builders: v8_presubmit on tryserver.v8 (JOB_FAILED, http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/9100)
5 years ago (2015-12-17 13:20:27 UTC) #7
Camillo Bruni
lgtm
5 years ago (2015-12-17 13:26:27 UTC) #9
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1531243003/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1531243003/40001
5 years ago (2015-12-17 13:27:07 UTC) #11
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1531243003/40001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1531243003/40001
5 years ago (2015-12-17 13:29:30 UTC) #14
commit-bot: I haz the power
Try jobs failed on following builders: v8_presubmit on tryserver.v8 (JOB_FAILED, http://build.chromium.org/p/tryserver.v8/builders/v8_presubmit/builds/9103)
5 years ago (2015-12-17 13:31:59 UTC) #16
ahaas
lgtm
5 years ago (2015-12-17 13:51:02 UTC) #17
Jarin
lgtm (the Realloc stuff is terrible, but bradn promised to make sure this is cleaned ...
5 years ago (2015-12-17 14:45:21 UTC) #19
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1531243003/60001 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1531243003/60001
5 years ago (2015-12-17 14:45:56 UTC) #22
commit-bot: I haz the power
Committed patchset #4 (id:60001)
5 years ago (2015-12-17 14:52:44 UTC) #24
commit-bot: I haz the power
5 years ago (2015-12-17 14:52:55 UTC) #26
Message was sent while issue was closed.
Patchset 4 (id:??) landed as
https://crrev.com/7b775118b98eba613ab3a5936f9c489ed7d1b45a
Cr-Commit-Position: refs/heads/master@{#32951}

Powered by Google App Engine
This is Rietveld 408576698