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

Unified Diff: build/standalone.gypi

Issue 1406843002: Add experimental support for compiling with clang and ASan on Linux. (Closed) Base URL: https://pdfium.googlesource.com/pdfium.git@master
Patch Set: remove unused var Created 5 years, 2 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 | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: build/standalone.gypi
diff --git a/build/standalone.gypi b/build/standalone.gypi
index c0eaef419df30e2ff6538aed44e0940d87aa128d..34718d18de6a3cf950a4a754a6a882a112068ec1 100644
--- a/build/standalone.gypi
+++ b/build/standalone.gypi
@@ -7,6 +7,7 @@
{
'variables': {
'component%': 'static_library',
+ 'asan%': 0,
'clang%': 0,
'msvs_multi_core_compile%': '1',
'variables': {
@@ -161,6 +162,17 @@
'ldflags': [
'-pthread',
],
+ 'conditions': [
+ ['asan==1', {
+ 'cflags': [
+ '-fsanitize=address',
+ '-gline-tables-only',
+ ],
+ 'ldflags': [
+ '-fsanitize=address',
+ ],
+ }],
+ ],
'defines': [
# Don't use deprecated V8 APIs anywhere.
'V8_DEPRECATION_WARNINGS',
@@ -250,6 +262,16 @@
'-fPIC',
],
}],
+ ['OS=="linux"', {
+ 'conditions': [
+ ['clang==1', {
+ 'make_global_settings': [
+ ['CC', '<!(which clang)'],
Nico 2015/10/14 23:55:21 Isn't it better if you deps in chromium's tools/cl
Oliver Chang 2015/10/15 01:00:49 Ah I didn't know v8 does this, thanks! I got a lit
+ ['CXX', '<!(which clang++)'],
+ ],
+ }],
+ ],
+ }], # OS=="linux"
['OS=="win"', {
'target_defaults': {
'defines': [
« no previous file with comments | « no previous file | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698