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

Issue 18431003: Add a 'v8_optimized_debug' option to the build flags (Closed)

Created:
7 years, 5 months ago by Dirk Pranke
Modified:
7 years, 5 months ago
CC:
v8-dev
Visibility:
Public.

Description

PATCH ABANDONED ... see https://codereview.chromium.org/19384011/ Add a 'v8_optimized_debug' option to the build flags This allows us to do a build with the DEBUG macros enabled and with all of the compiler optimizations still happening. This provides a speedup of 3x - 4x when running the layout tests in chromium/blink (this makes the debug builds only 3x-4x slower than Release, as opposed to 10x-11x slower). R=danno@chromium.org BUG=https://code.google.com/p/chromium/issues/detail?id=254188

Patch Set 1 #

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

Messages

Total messages: 5 (0 generated)
Dirk Pranke
7 years, 5 months ago (2013-07-09 23:56:26 UTC) #1
Michael Achenbach
I cannot get the code to compile with gcc under linux64. I get weird array ...
7 years, 5 months ago (2013-07-10 13:19:31 UTC) #2
Dirk Pranke
On 2013/07/10 13:19:31, machenbach wrote: > I cannot get the code to compile with gcc ...
7 years, 5 months ago (2013-07-10 17:00:13 UTC) #3
Michael Achenbach
On 2013/07/10 17:00:13, Dirk Pranke wrote: > On 2013/07/10 13:19:31, machenbach wrote: > > I ...
7 years, 5 months ago (2013-07-10 18:48:40 UTC) #4
Michael Achenbach
7 years, 5 months ago (2013-07-12 09:58:16 UTC) #5
We are building debug now with the -O1 optimization. See also last comment in
crbug.com/238095

Powered by Google App Engine
This is Rietveld 408576698