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

Side by Side Diff: runtime/BUILD.gn

Issue 922863003: Allows gen_snapshot to be built with asan in the Mojo tree. (Closed) Base URL: http://dart.googlecode.com/svn/branches/bleeding_edge/dart/
Patch Set: Created 5 years, 10 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 unified diff | Download patch | Annotate | Revision Log
« no previous file with comments | « no previous file | runtime/bin/BUILD.gn » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 # Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file 1 # Copyright (c) 2014, the Dart project authors. Please see the AUTHORS file
2 # for details. All rights reserved. Use of this source code is governed by a 2 # for details. All rights reserved. Use of this source code is governed by a
3 # BSD-style license that can be found in the LICENSE file. 3 # BSD-style license that can be found in the LICENSE file.
4 4
5 # TODO(zra): These build arguments should likely be moved to a gni file that is 5 # TODO(zra): These build arguments should likely be moved to a gni file that is
6 # included in BUILD.gn files that care about the values of the flags. For now, 6 # included in BUILD.gn files that care about the values of the flags. For now,
7 # since the GN build only happens as part of a Mojo build there is no need for 7 # since the GN build only happens as part of a Mojo build there is no need for
8 # the indirection. 8 # the indirection.
9 declare_args() { 9 declare_args() {
10 # Instead of using is_debug, we introduce a different flag for specifying a 10 # Instead of using is_debug, we introduce a different flag for specifying a
(...skipping 27 matching lines...) Expand all
38 38
39 if (dart_debug) { 39 if (dart_debug) {
40 cflags += [ 40 cflags += [
41 "-O1", 41 "-O1",
42 ] 42 ]
43 } else { 43 } else {
44 cflags += [ 44 cflags += [
45 "-O3", 45 "-O3",
46 ] 46 ]
47 } 47 }
48
49 if (is_asan) {
50 ldflags = [
51 "-Wl,-u_sanitizer_options_link_helper",
52 "-fsanitize=address",
53 ]
54 }
48 } 55 }
49 56
50 57
51 static_library("libdart") { 58 static_library("libdart") {
52 configs += [":dart_config"] 59 configs += [":dart_config"]
53 deps = [ 60 deps = [
54 "vm:libdart_lib", 61 "vm:libdart_lib",
55 "vm:libdart_vm", 62 "vm:libdart_vm",
56 "third_party/jscre:libjscre", 63 "third_party/jscre:libjscre",
57 "third_party/double-conversion/src:libdouble_conversion", 64 "third_party/double-conversion/src:libdouble_conversion",
(...skipping 51 matching lines...) Expand 10 before | Expand all | Expand 10 after
109 "vm:libdart_lib", 116 "vm:libdart_lib",
110 "vm:libdart_vm", 117 "vm:libdart_vm",
111 "vm:libdart_platform", 118 "vm:libdart_platform",
112 "third_party/jscre:libjscre", 119 "third_party/jscre:libjscre",
113 "third_party/double-conversion/src:libdouble_conversion", 120 "third_party/double-conversion/src:libdouble_conversion",
114 ] 121 ]
115 sources = [ 122 sources = [
116 "vm/libdart_dependency_helper.cc", 123 "vm/libdart_dependency_helper.cc",
117 ] 124 ]
118 } 125 }
OLDNEW
« no previous file with comments | « no previous file | runtime/bin/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698