Reviewers: Kevin Millikin,

Message:
PTAL.

Description:
Merge revision 8899 from bleeding_edge

Third attempt to fix ARM cross-compile build


Please review this at http://codereview.chromium.org/7617009/

SVN Base: https://v8.googlecode.com/svn/trunk

Affected files:
  M src/version.cc
  M tools/gyp/v8.gyp


Index: src/version.cc
diff --git a/src/version.cc b/src/version.cc
index f056458dbdc921a9315473774b933731853e18d3..83836f6207e6c65766c2f20d9b0f73918a553543 100644
--- a/src/version.cc
+++ b/src/version.cc
@@ -35,7 +35,7 @@
 #define MAJOR_VERSION     3
 #define MINOR_VERSION     5
 #define BUILD_NUMBER      4
-#define PATCH_LEVEL       2
+#define PATCH_LEVEL       3
 // Use 1 for candidates and 0 otherwise.
 // (Boolean macro values are not supported by all preprocessors.)
 #define IS_CANDIDATE_VERSION 0
Index: tools/gyp/v8.gyp
diff --git a/tools/gyp/v8.gyp b/tools/gyp/v8.gyp
index 7a36d823675ace56068607c3e403057ae1e13890..e755fcb926b5b47e1a73db99c172fa418ee3dc18 100644
--- a/tools/gyp/v8.gyp
+++ b/tools/gyp/v8.gyp
@@ -319,10 +319,13 @@
             }],
             # The ARM assembler assumes the host is 32 bits,
             # so force building 32-bit host tools.
-            ['v8_target_arch=="arm" and host_arch=="x64" and \
-              _toolset=="host"', {
-              'cflags': ['-m32'],
-              'ldflags': ['-m32'],
+            ['v8_target_arch=="arm" and host_arch=="x64"', {
+              'target_conditions': [
+                ['_toolset=="host"', {
+                  'cflags': ['-m32'],
+                  'ldflags': ['-m32'],
+                }],
+              ],
             }],
             ['component=="shared_library"', {
               'conditions': [
@@ -429,10 +432,13 @@
           'conditions': [
             # The ARM assembler assumes the host is 32 bits,
             # so force building 32-bit host tools.
-            ['v8_target_arch=="arm" and host_arch=="x64" and \
-              _toolset=="host"', {
-              'cflags': ['-m32'],
-              'ldflags': ['-m32'],
+            ['v8_target_arch=="arm" and host_arch=="x64"', {
+              'target_conditions': [
+                ['_toolset=="host"', {
+                  'cflags': ['-m32'],
+                  'ldflags': ['-m32'],
+                }],
+              ],
             }],
             ['want_separate_host_toolset==1', {
               'toolsets': ['host', 'target'],
@@ -748,9 +754,13 @@
               'conditions': [
                 # The ARM assembler assumes the host is 32 bits,
                 # so force building 32-bit host tools.
-                ['host_arch=="x64" and _toolset=="host"', {
-                  'cflags': ['-m32'],
-                  'ldflags': ['-m32'],
+                ['host_arch=="x64"', {
+                  'target_conditions': [
+                    ['_toolset=="host"', {
+                      'cflags': ['-m32'],
+                      'ldflags': ['-m32'],
+                    }],
+                  ],
                 }],
               ],
             }],
@@ -993,10 +1003,13 @@
             }],
             # The ARM assembler assumes the host is 32 bits,
             # so force building 32-bit host tools.
-            ['v8_target_arch=="arm" and host_arch=="x64" and \
-              _toolset=="host"', {
-              'cflags': ['-m32'],
-              'ldflags': ['-m32'],
+            ['v8_target_arch=="arm" and host_arch=="x64"', {
+              'target_conditions': [
+                ['_toolset=="host"', {
+                  'cflags': ['-m32'],
+                  'ldflags': ['-m32'],
+                }],
+              ],
             }],
             ['v8_compress_startup_data=="bz2"', {
               'libraries': [
@@ -1065,10 +1078,13 @@
             }],
             # The ARM assembler assumes the host is 32 bits,
             # so force building 32-bit host tools.
-            ['v8_target_arch=="arm" and host_arch=="x64" and \
-              _toolset=="host"', {
-              'cflags': ['-m32'],
-              'ldflags': ['-m32'],
+            ['v8_target_arch=="arm" and host_arch=="x64"', {
+              'target_conditions': [
+                ['_toolset=="host"', {
+                  'cflags': ['-m32'],
+                  'ldflags': ['-m32'],
+                }],
+              ],
             }],
           ],
           'link_settings': {


--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to