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

Unified Diff: Makefile

Issue 847753002: Merge v8_optimized_debug 1 and 2 (Closed) Base URL: https://chromium.googlesource.com/v8/v8.git@master
Patch Set: Created 5 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « no previous file | build/standalone.gypi » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: Makefile
diff --git a/Makefile b/Makefile
index 6c2955126bdf3e99b9547336de388ac46bbd1fa4..cb7ad7136b74c79e446e436b3d487d81331429b9 100644
--- a/Makefile
+++ b/Makefile
@@ -110,7 +110,7 @@ ifeq ($(vtunejit), on)
endif
# optdebug=on
ifeq ($(optdebug), on)
Michael Achenbach 2015/01/12 08:55:42 Wonder if this is necessary. We have two modes alr
- GYPFLAGS += -Dv8_optimized_debug=2
+ GYPFLAGS += -Dv8_optimized_debug=1
endif
# unalignedaccess=on
ifeq ($(unalignedaccess), on)
@@ -430,7 +430,7 @@ $(OUT_MAKEFILES): $(GYPFILES) $(ENVFILE)
-Dv8_target_arch=$(V8_TARGET_ARCH) \
$(if $(findstring $(CXX_TARGET_ARCH),$(V8_TARGET_ARCH)), \
-Dtarget_arch=$(V8_TARGET_ARCH),) \
- $(if $(findstring optdebug,$@),-Dv8_optimized_debug=2,) \
+ $(if $(findstring optdebug,$@),-Dv8_optimized_debug=1,) \
Michael Achenbach 2015/01/12 08:55:42 nit: Can you maybe get rid of the tabs?
-S$(suffix $(basename $@))$(suffix $@) $(GYPFLAGS)
$(OUTDIR)/Makefile.native: $(GYPFILES) $(ENVFILE)
« no previous file with comments | « no previous file | build/standalone.gypi » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698