[MIPS] Add support to gyp files to build Native Client inside of Chromium

build/common.gypi

 # Copyright (c) 2012 The Chromium Authors. All rights reserved.
 # Use of this source code is governed by a BSD-style license that can be
 # found in the LICENSE file.
 
 {
   'variables': {
     # .gyp files should set chromium_code to 1 if they build Chromium-specific
     # code, as opposed to external code.  This variable is used to control
     # such things as the set of warnings to enable, and whether warnings are
     # treated as errors.
@@ skipping 186 matching lines @@
         ],
       }],
       ['target_arch=="arm"', {
         'defines': [
           'NACL_BUILD_ARCH=arm',
           'NACL_BUILD_SUBARCH=32',
           'NACL_TARGET_ARCH=arm',
           'NACL_TARGET_SUBARCH=32',
         ],
       }],
+      ['target_arch=="mipsel"', {
+        'defines': [
+          'NACL_BUILD_ARCH=mipsel',

[Mark Seaborn, 2013/09/20 23:11:17]

Why not "mips" instead of "mipsel"?  Then we don't need the nacl_base.h change.

[petarj, 2013/09/23 17:49:24]

Done.

+          'NACL_BUILD_SUBARCH=32',
+          'NACL_TARGET_ARCH=mipsel',
+          'NACL_TARGET_SUBARCH=32',
+        ],
+      }],
       ['linux2==1', {
         'defines': ['LINUX2=1'],
       }],
       ['OS=="win"', {
         'defines': [ 'NOMINMAX' ]
       }],
       ['coverage!=0', {
         'conditions': [
           ['OS=="mac"', {
             'xcode_settings': {
@@ skipping 113 matching lines @@
                       ],
                       'conditions': [
                           ['arm_neon==1', {
                               'cflags': [ '-mfpu=neon', ],
                           }, {
                               'cflags': [ '-mfpu=<(arm_fpu)', ],
                           }]
                       ],
                   }],
               ],
-            }, { # else: target_arch != "arm"
+            }],
+            ['target_arch=="mipsel"', {
+              # Copied from chromium build/common.gypi
+              'conditions': [
+                ['mips_arch_variant=="mips32r2"', {
+                    'cflags': ['-mips32r2'],

[Mark Seaborn, 2013/09/20 23:11:17]

Indent by 1 fewer space (i.e. 2 spaces per indentation level)

[petarj, 2013/09/23 17:49:24]

Done.

+                }, {
+                    'cflags': ['-mips32'],

[Mark Seaborn, 2013/09/20 23:11:17]

Same here

[petarj, 2013/09/23 17:49:24]

Done.

+                }],
+              ],
+            }],
+            ['target_arch=="ia32" or target_arch=="x64"', {
               'conditions': [
                 ['target_arch=="x64"', {
                   'variables': {
                     'mbits_flag': '-m64',
                   },
                 }, {
                   'variables': {
                     'mbits_flag': '-m32',
                   }
                 },],
@@ skipping 494 matching lines @@
     # like PROJECT_DERIVED_FILE_DIR is added to a project, in order for the
     # files to appear (when present) in the UI as actual files and not red
     # red "missing file" proxies, the correct path to PROJECT_DERIVED_FILE_DIR,
     # and therefore SYMROOT, needs to be set at the project level.
     'SYMROOT': '<(DEPTH)/xcodebuild',
   },
   'includes': [
     'untrusted.gypi',
   ],
 }