Issue 1211513002: Re-land new insertion write barrier.

Issue 1211513002: Re-land new insertion write barrier. (Closed)

Created:
5 years, 6 months ago by Hannes Payer (out of office)

Modified:
5 years, 6 months ago

Reviewers:
ulan

CC:
v8-dev

Base URL:
https://chromium.googlesource.com/v8/v8.git@master

Target Ref:
refs/pending/heads/master

Project:
v8

Visibility:
Public.

More Reviews

Description

Re-land new insertion write barrier. BUG= Committed: https://crrev.com/6e6af7e7c70050cdd5ce56cdbaee3b6e8a7caa89 Cr-Commit-Position: refs/heads/master@{#29258}

Patch Set 1 #

Created: 5 years, 6 months ago

Download [raw] [tar.bz2]

		Unified diffs	Side-by-side diffs	Delta from patch set	Stats (+6 lines, -27 lines)			Patch
	M	src/heap/incremental-marking-inl.h	View		1 chunk	+6 lines, -20 lines	0 comments	Download
	M	src/heap/spaces.h	View		1 chunk	+0 lines, -7 lines	0 comments	Download

Messages

Total messages: 8 (2 generated)

Expand Messages | Collapse Messages | Show Generated Messages | Hide Generated Messages

commit-bot: I haz the power

CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1211513002/1

5 years, 6 months ago (2015-06-24 16:12:39 UTC) #5

commit-bot: I haz the power

Patchset 1 (id:??) landed as https://crrev.com/6e6af7e7c70050cdd5ce56cdbaee3b6e8a7caa89 Cr-Commit-Position: refs/heads/master@{#29258}

5 years, 6 months ago (2015-06-24 16:41:03 UTC) #7

Hannes Payer (out of office)

5 years, 6 months ago (2015-06-26 10:52:37 UTC) #8

Message was sent while issue was closed.

A revert of this CL (patchset #1 id:1) has been created in
https://codereview.chromium.org/1217443002/ by hpayer@chromium.org.

The reason for reverting is: Still crashing in the wild..

Expand Messages | Collapse Messages | Show Generated Messages | Hide Generated Messages