Skip to content

Commit

Permalink
[sandbox] Update test/mkgrokdump/README
Browse files Browse the repository at this point in the history
When regenerating v8heapconst.py, the v8 sandbox now has to be enabled
explicitly (using the v8_enabe_sandbox=true gn arg) as it is enabled by
default in Chromium builds, but not standalone v8 builds.

Bug: v8:13281
Change-Id: I1a0861b1d63f340465d7433e042b27eace706ca7
Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3895622
Commit-Queue: Adam Klein <adamk@chromium.org>
Auto-Submit: Samuel Groß <saelo@chromium.org>
Reviewed-by: Adam Klein <adamk@chromium.org>
Cr-Commit-Position: refs/heads/main@{#83201}
  • Loading branch information
Samuel Groß authored and V8 LUCI CQ committed Sep 14, 2022
1 parent 429f959 commit 9521696
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions test/mkgrokdump/README
Original file line number Diff line number Diff line change
@@ -1,3 +1,5 @@
If you change the heap layout, expect this test to fail (in CQ). You will then
need to build x64 release and run `<outdir>/mkgrokdump > tools/v8heapconst.py`
to rebaseline.
need to build x64 release with v8_enable_sandbox=true and run
`<outdir>/mkgrokdump > tools/v8heapconst.py` to rebaseline.
The V8 sandbox has to be manually enabled as it is enabled by default only for
Chromium builds, but not standalone v8 builds.

0 comments on commit 9521696

Please sign in to comment.