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

Side by Side Diff: webrtc/BUILD.gn

Issue 3007153003: Enable UBSan float-cast-overflow warnings and fix existing ones (Closed)
Patch Set: Fix/silence existing warnings Created 3 years, 3 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
OLDNEW
1 # Copyright (c) 2014 The WebRTC project authors. All Rights Reserved. 1 # Copyright (c) 2014 The WebRTC project authors. All Rights Reserved.
2 # 2 #
3 # Use of this source code is governed by a BSD-style license 3 # Use of this source code is governed by a BSD-style license
4 # that can be found in the LICENSE file in the root of the source 4 # that can be found in the LICENSE file in the root of the source
5 # tree. An additional intellectual property rights grant can be found 5 # tree. An additional intellectual property rights grant can be found
6 # in the file PATENTS. All contributing project authors may 6 # in the file PATENTS. All contributing project authors may
7 # be found in the AUTHORS file in the root of the source tree. 7 # be found in the AUTHORS file in the root of the source tree.
8 8
9 # TODO(kjellander): Rebase this to webrtc/build/common.gypi changes after r6330. 9 # TODO(kjellander): Rebase this to webrtc/build/common.gypi changes after r6330.
10 10
(...skipping 75 matching lines...) Expand 10 before | Expand all | Expand 10 after
86 if (is_chromeos) { 86 if (is_chromeos) {
87 defines += [ "CHROMEOS" ] 87 defines += [ "CHROMEOS" ]
88 } 88 }
89 89
90 if (rtc_sanitize_coverage != "") { 90 if (rtc_sanitize_coverage != "") {
91 assert(is_clang, "sanitizer coverage requires clang") 91 assert(is_clang, "sanitizer coverage requires clang")
92 cflags += [ "-fsanitize-coverage=${rtc_sanitize_coverage}" ] 92 cflags += [ "-fsanitize-coverage=${rtc_sanitize_coverage}" ]
93 ldflags += [ "-fsanitize-coverage=${rtc_sanitize_coverage}" ] 93 ldflags += [ "-fsanitize-coverage=${rtc_sanitize_coverage}" ]
94 } 94 }
95 95
96 if (is_ubsan) {
97 cflags += [ "-fsanitize=float-cast-overflow" ]
98 }
99
96 # TODO(GYP): Support these in GN. 100 # TODO(GYP): Support these in GN.
97 # if (is_bsd) { 101 # if (is_bsd) {
98 # defines += [ "BSD" ] 102 # defines += [ "BSD" ]
99 # } 103 # }
100 # if (is_openbsd) { 104 # if (is_openbsd) {
101 # defines += [ "OPENBSD" ] 105 # defines += [ "OPENBSD" ]
102 # } 106 # }
103 # if (is_freebsd) { 107 # if (is_freebsd) {
104 # defines += [ "FREEBSD" ] 108 # defines += [ "FREEBSD" ]
105 # } 109 # }
(...skipping 425 matching lines...) Expand 10 before | Expand all | Expand 10 after
531 ] 535 ]
532 536
533 deps = [ 537 deps = [
534 "examples:AppRTCMobile_javalib", 538 "examples:AppRTCMobile_javalib",
535 "sdk/android:libjingle_peerconnection_java", 539 "sdk/android:libjingle_peerconnection_java",
536 "//base:base_java_test_support", 540 "//base:base_java_test_support",
537 ] 541 ]
538 } 542 }
539 } 543 }
540 } 544 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698