update gyp for ARM builds to be more well defined.

gyp/common_conditions.gypi

Index: gyp/common_conditions.gypi
diff --git a/gyp/common_conditions.gypi b/gyp/common_conditions.gypi
index 6828660444874adf252dc37a1879985b269d3d3a..f254e66fa54d2e0c7a6d178c20cf758fa216489a 100644
--- a/gyp/common_conditions.gypi
+++ b/gyp/common_conditions.gypi
@@ -150,17 +150,30 @@
             'cflags': [
               '-mthumb',
             ],
-          }],
-          [ 'skia_arch_type == "arm" and armv7 == 1', {
-            'variables': {
-              'arm_neon_optional%': 0,
-            },
-            'defines': [
-              '__ARM_ARCH__=7',
+            # The --fix-cortex-a8 switch enables a link-time workaround for
+            # an erratum in certain Cortex-A8 processors.  The workaround is
+            # enabled by default if you target the ARM v7-A arch profile.
+            # It can be enabled otherwise by specifying --fix-cortex-a8, or
+            # disabled unconditionally by specifying --no-fix-cortex-a8.
+            #
+            # The erratum only affects Thumb-2 code.
+            'conditions': [
+              [ 'arm_version < 7', {
+                'ldflags': [
+                  '-Wl,--fix-cortex-a8',
+                ],
+              }],
             ],
+          }],
+          [ 'skia_arch_type == "arm" and arm_version == 7', {

[mtklein, 2013/07/29 20:02:16]

Merge this back with below for now?  Otherwise we won't have -march=anything for
arm_version > 7.  Seems better to leave it as armv7-a until we increase it
further.

[djsollen, 2013/07/29 20:25:59]

The reason I moved it out is that there is not always a guarantee that the next
version will be backwards compatible. In the case of v8 it should be fine so I
can move it back.

             'cflags': [
               '-march=armv7-a',
             ],
+            'ldflags': [
+              '-march=armv7-a',
+            ],
+          }],
+          [ 'skia_arch_type == "arm" and arm_version >= 7', {
             'conditions': [
               [ 'arm_neon == 1', {
                 'defines': [
@@ -169,10 +182,6 @@
                 'cflags': [
                   '-mfpu=neon',
                 ],
-                'ldflags': [
-                  '-march=armv7-a',
-                  '-Wl,--fix-cortex-a8',
-                ],
               }],
               [ 'arm_neon_optional == 1', {
                 'defines': [