MIPS compiler flags: use -mfp32 or -mfp64 only if -mhard-float is used.

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.
 
 # IMPORTANT:
 # Please don't directly include this file if you are building via gyp_chromium,
 # since gyp_chromium is automatically forcing its inclusion.
 {
   # Variables expected to be overriden on the GYP command line (-D) or by
   # ~/.gyp/include.gypi.
@@ skipping 2284 matching lines @@
             'arm_fpu%': 'neon',
           }, {
             'arm_fpu%': 'vfpv3-d16',
           }],
         ],
         # Change the default to hard once the armhf transition is complete.
         'arm_float_abi%': 'softfp',
         'arm_thumb%': 1,
       }],
 
-      # Set default compiler flags depending on MIPS architecture variant.
+      # Set default compiler flags for MIPS floating-point support.
+      ['target_arch=="mipsel" and android_webview_build==0', {
+        'mips_float_abi%': 'hard',
+      }],
       ['target_arch=="mipsel" and mips_arch_variant=="r2" and android_webview_build==0', {
         'mips_fpu_mode%': 'fp32',

[wtc, 2015/03/03 23:21:41]

I reordered these two build variables to suggest that mips_fpu_mode is
subordinate to mips_float_abi.

       }],
-      ['target_arch=="mipsel" and android_webview_build==0', {
-        'mips_float_abi%': 'hard',
-      }],
 
       ['android_webview_build==1', {
         # The WebView build gets its cpu-specific flags from the Android build system.
         'arm_arch%': '',
         'arm_tune%': '',
         'arm_fpu%': '',
         'arm_float_abi%': '',
         'arm_thumb%': 0,
+        'mips_float_abi%': '',
         'mips_fpu_mode%': '',
-        'mips_float_abi%': '',
       }],
 
       # Enable brlapi by default for chromeos.
       [ 'chromeos==1', {
         'use_brlapi%': 1,
       }],
 
       ['use_ozone==1 and ozone_auto_platforms==1', {
         # Use test as the default platform.
         'ozone_platform%': 'test',
@@ skipping 1742 matching lines @@
                         'cflags': ['-mips32r6', '-Wa,-mips32r6'],
                         'conditions': [
                           ['OS=="android"', {
                             'ldflags': ['-mips32r6', '-Wl,-melf32ltsmip',],
                           }],
                         ],
                       }],
                       ['mips_arch_variant=="r2"', {
                         'cflags': ['-mips32r2', '-Wa,-mips32r2'],
                         'conditions': [
-                          ['mips_fpu_mode!=""', {
+                          ['mips_float_abi=="hard" and mips_fpu_mode!=""', {
                             'cflags': ['-m<(mips_fpu_mode)'],
                           }],
                         ],
                       }],
                       ['mips_arch_variant=="r1"', {
                         'cflags': ['-mips32', '-Wa,-mips32'],
                       }],
                       ['mips_dsp_rev==1', {
                         'cflags': ['-mdsp'],
                       }],
@@ skipping 1884 matching lines @@
     # settings in target dicts.  SYMROOT is a special case, because many other
     # Xcode variables depend on it, including variables such as
     # PROJECT_DERIVED_FILE_DIR.  When a source group corresponding to something
     # 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',
   },
 }