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

Side by Side Diff: third_party/libc++/BUILD.gn

Issue 2897463004: libc++: change default symbol visibility to hidden (Closed)
Patch Set: Update libc++abi as well Created 3 years, 7 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
« no previous file with comments | « no previous file | third_party/libc++abi/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 2015 The Chromium Authors. All rights reserved. 1 # Copyright 2015 The Chromium Authors. All rights reserved.
2 # Use of this source code is governed by a BSD-style license that can be 2 # Use of this source code is governed by a BSD-style license that can be
3 # found in the LICENSE file. 3 # found in the LICENSE file.
4 4
5 import("//build/config/sanitizers/sanitizers.gni") 5 import("//build/config/sanitizers/sanitizers.gni")
6 import("//build/toolchain/toolchain.gni") 6 import("//build/toolchain/toolchain.gni")
7 7
8 # Used by libc++ and libc++abi. 8 # Used by libc++ and libc++abi.
9 config("config") { 9 config("config") {
10 defines = [ "LIBCXX_BUILDING_LIBCXXABI" ] 10 defines = [ "LIBCXX_BUILDING_LIBCXXABI" ]
(...skipping 37 matching lines...) Expand 10 before | Expand all | Expand 10 after
48 "trunk/src/system_error.cpp", 48 "trunk/src/system_error.cpp",
49 "trunk/src/thread.cpp", 49 "trunk/src/thread.cpp",
50 "trunk/src/typeinfo.cpp", 50 "trunk/src/typeinfo.cpp",
51 "trunk/src/utility.cpp", 51 "trunk/src/utility.cpp",
52 "trunk/src/valarray.cpp", 52 "trunk/src/valarray.cpp",
53 ] 53 ]
54 configs -= [ 54 configs -= [
55 "//build/config/compiler:chromium_code", 55 "//build/config/compiler:chromium_code",
56 "//build/config/compiler:no_rtti", 56 "//build/config/compiler:no_rtti",
57 "//build/config/gcc:no_exceptions", 57 "//build/config/gcc:no_exceptions",
58 "//build/config/gcc:symbol_visibility_hidden",
59 ] 58 ]
60 configs += [ 59 configs += [
61 ":config", 60 ":config",
62 "//build/config/compiler:no_chromium_code", 61 "//build/config/compiler:no_chromium_code",
63 "//build/config/compiler:rtti", 62 "//build/config/compiler:rtti",
64 "//build/config/gcc:symbol_visibility_default",
65 "//build/config/sanitizers:sanitizer_options_link_helper", 63 "//build/config/sanitizers:sanitizer_options_link_helper",
66 ] 64 ]
67 65
68 ldflags = [ "-nodefaultlibs" ] 66 ldflags = [ "-nodefaultlibs" ]
69 67
70 # Remove "-pthread" from ldflags. 68 # Remove "-pthread" from ldflags.
71 # -nodefaultlibs turns -pthread into a no-op, causing an unused argument 69 # -nodefaultlibs turns -pthread into a no-op, causing an unused argument
72 # warning. Explicitly link with -lpthread instead. 70 # warning. Explicitly link with -lpthread instead.
73 configs -= [ "//build/config/compiler:pthread" ] 71 configs -= [ "//build/config/compiler:pthread" ]
74 72
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
123 # Normally the generator takes care of RPATH. Our case is special because 121 # Normally the generator takes care of RPATH. Our case is special because
124 # the generator is unaware of the libc++.so dependency. Note that setting 122 # the generator is unaware of the libc++.so dependency. Note that setting
125 # RPATH here is a potential security issue. See the following for another 123 # RPATH here is a potential security issue. See the following for another
126 # example of this issue: https://code.google.com/p/gyp/issues/detail?id=31 5 124 # example of this issue: https://code.google.com/p/gyp/issues/detail?id=31 5
127 "-Wl,-rpath,\$ORIGIN/", 125 "-Wl,-rpath,\$ORIGIN/",
128 ] 126 ]
129 } 127 }
130 128
131 lib_dirs = [ root_build_dir ] 129 lib_dirs = [ root_build_dir ]
132 } 130 }
OLDNEW
« no previous file with comments | « no previous file | third_party/libc++abi/BUILD.gn » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698