-
Notifications
You must be signed in to change notification settings - Fork 0
/
0001-Drop-_FORTIFY_SOURCE-2-from-defines.patch
39 lines (35 loc) · 1.47 KB
/
0001-Drop-_FORTIFY_SOURCE-2-from-defines.patch
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
From 2948f06c7995d54d87edc6df16ea77ea03dc6b4b Mon Sep 17 00:00:00 2001
From: Pietro Di Gennaro <[email protected]>
Date: Sun, 11 Mar 2018 16:16:41 +0100
Subject: [PATCH] Drop _FORTIFY_SOURCE=2 from defines
---
build/config/compiler/BUILD.gn | 16 ----------------
1 file changed, 16 deletions(-)
diff --git a/build/config/compiler/BUILD.gn b/build/config/compiler/BUILD.gn
index 5c1e34350ac7..09f1c76f365a 100644
--- a/build/config/compiler/BUILD.gn
+++ b/build/config/compiler/BUILD.gn
@@ -1382,22 +1382,6 @@ config("chromium_code") {
"__STDC_FORMAT_MACROS",
]
- if (!is_debug && !using_sanitizer &&
- (!is_linux || !is_clang || is_official_build) &&
- current_cpu != "s390x" && current_cpu != "s390" &&
- current_cpu != "ppc64" && current_cpu != "ppc64" &&
- current_cpu != "mips" && current_cpu != "mips64") {
- # _FORTIFY_SOURCE isn't really supported by Clang now, see
- # http://llvm.org/bugs/show_bug.cgi?id=16821.
- # It seems to work fine with Ubuntu 12 headers though, so use it in
- # official builds.
- #
- # Non-chromium code is not guaranteed to compile cleanly with
- # _FORTIFY_SOURCE. Also, fortified build may fail when optimizations are
- # disabled, so only do that for Release build.
- defines += [ "_FORTIFY_SOURCE=2" ]
- }
-
if (is_mac || is_ios) {
cflags_objc = [ "-Wobjc-missing-property-synthesis" ]
cflags_objcc = [ "-Wobjc-missing-property-synthesis" ]
--
2.14.3