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

Issue 11275324: Disable EXTRA_CHECKS in Release (Closed)

Created:
8 years, 1 month ago by abarth-chromium
Modified:
8 years, 1 month ago
CC:
v8-dev, haraken, tony, danno
Base URL:
git://github.com/v8/v8.git@bleeding_edge
Visibility:
Public.

Description

Disable EXTRA_CHECKS in Release This patch causes V8 to disable EXTRA_CHECKS when building in release. You can still enable the checks in release using a GYP flag. This patch speeds up Dromeo's dom-traverse by around 4%. Committed: https://code.google.com/p/v8/source/detail?r=12981

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -6 lines) Patch
M build/common.gypi View 5 chunks +12 lines, -6 lines 0 comments Download

Messages

Total messages: 6 (0 generated)
abarth-chromium
This is a simple patch that makes dom-traverse around 4% faster on my Mac. I'm ...
8 years, 1 month ago (2012-11-14 22:25:29 UTC) #1
Sven Panne
[Looping in our GYP master, too] EXTRA_CHECKS was introduced mainly to catch bugs in native ...
8 years, 1 month ago (2012-11-15 06:51:28 UTC) #2
abarth-chromium
The release managers don't like shipping different builds to different channels because it makes the ...
8 years, 1 month ago (2012-11-15 08:41:13 UTC) #3
danno
We landed the extra checks with the assumption that they wouldn't negatively impact perf (at ...
8 years, 1 month ago (2012-11-15 08:57:18 UTC) #4
abarth-chromium
If the API abuse comes back, we can turn them back on to try and ...
8 years, 1 month ago (2012-11-15 16:44:00 UTC) #5
Jakob Kummerow
8 years, 1 month ago (2012-11-15 17:01:01 UTC) #6
LGTM.

I would've liked to keep these checks in for ToT/Canary builds, but I can
understand that the Release team would prefer to avoid that.

I'll land this.

Powered by Google App Engine
This is Rietveld 408576698