diff options
author | scroggo <scroggo@google.com> | 2015-02-12 10:48:25 -0800 |
---|---|---|
committer | Commit bot <commit-bot@chromium.org> | 2015-02-12 10:48:25 -0800 |
commit | df1c3373fc198f465cb0077516993dd41f268416 (patch) | |
tree | 5df85329712ed32721983b72a8db3f1e474899c6 /gyp/common_variables.gypi | |
parent | 9d7ceca35064b71e13e498da7e641e69b97bf4cf (diff) |
Don't use m32 cflag for x86_64.
When checking the skia_arch_type for "x86", instead of doing an
== compare, check if "x86" in skia_arch_type, so it will cover
both x86 and x86_64.
Except when we specifically want x86.
Set skia_arch_width based on "64" in skia_arch_type. No need to specify
in scripts.
In gyp_to_android.py, create a separate var_dict for x86_64.
BUG=skia:3419
Review URL: https://codereview.chromium.org/916113002
Diffstat (limited to 'gyp/common_variables.gypi')
-rw-r--r-- | gyp/common_variables.gypi | 23 |
1 files changed, 16 insertions, 7 deletions
diff --git a/gyp/common_variables.gypi b/gyp/common_variables.gypi index f89c08b95e..a20550b08d 100644 --- a/gyp/common_variables.gypi +++ b/gyp/common_variables.gypi @@ -40,14 +40,23 @@ # Variables needed by conditions list within the level-2 variables dict. 'variables': { # level 3 - # We use 'skia_os' instead of 'OS' throughout our gyp files, to allow - # for cross-compilation (e.g. building for either MacOS or iOS on Mac). - # We set it automatically based on 'OS' (the host OS), but allow the - # user to override it via GYP_DEFINES if they like. - 'skia_os%': '<(OS)', + 'variables': { # level 4 + # We use 'skia_os' instead of 'OS' throughout our gyp files, to allow + # for cross-compilation (e.g. building for either MacOS or iOS on Mac). + # We set it automatically based on 'OS' (the host OS), but allow the + # user to override it via GYP_DEFINES if they like. + 'skia_os%': '<(OS)', + }, + 'skia_os%': '<(skia_os)', 'skia_android_framework%': 0, - 'skia_arch_type%': 'x86', + 'conditions' : [ + [ 'skia_os in ["linux", "freebsd", "openbsd", "solaris", "mac"]', { + 'skia_arch_type%': 'x86_64', + }, { + 'skia_arch_type%': 'x86', + }], + ], 'arm_version%': 0, 'arm_neon%': 0, 'skia_egl%': 0, @@ -82,7 +91,7 @@ }, { 'skia_poppler_enabled%': 0, }], - [ 'skia_os in ["linux", "freebsd", "openbsd", "solaris", "mac"] or skia_arch_type == "arm64"', { + ['"64" in skia_arch_type', { 'skia_arch_width%': 64, }, { 'skia_arch_width%': 32, |