Only set /ARCH:IA32 on x86 platform

Only set /ARCH:IA32 on x86 platform

This eliminates compile warnings on x64 platform:

cl : Command line warning D9002 : ignoring unknown option '/arch:IA32'

BUG=374053
Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=271031

[scottmg, 2014-05-16 03:36:30]

lgtm

https://codereview.chromium.org/288353004/diff/1/chrome/chrome_installer.gypi
File chrome/chrome_installer.gypi (right):

https://codereview.chromium.org/288353004/diff/1/chrome/chrome_installer.gypi...
chrome/chrome_installer.gypi:357: [ 'target_arch=="ia32"', {
extra space after [

[Will Harris, 2014-05-16 03:45:26]

https://codereview.chromium.org/288353004/diff/1/chrome/chrome_installer.gypi
File chrome/chrome_installer.gypi (right):

https://codereview.chromium.org/288353004/diff/1/chrome/chrome_installer.gypi...
chrome/chrome_installer.gypi:357: [ 'target_arch=="ia32"', {
On 2014/05/16 03:36:30, scottmg wrote:
> extra space after [

Done.

oh the bad luck of the other gyp file I copied (mini_installer.gypi) having this
formatting! :)

[Will Harris, 2014-05-16 03:45:36]

The CQ bit was checked by wfh@chromium.org

[commit-bot: I haz the power, 2014-05-16 03:46:09]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/wfh@chromium.org/288353004/20001

[commit-bot: I haz the power, 2014-05-16 05:16:11]

FYI, CQ is re-trying this CL (attempt #1).
Please consider checking whether the failures are real,
and report flakes to chrome-troopers@google.com.
The failing builders are:
  linux_chromium_chromeos_rel on tryserver.chromium
(http://build.chromium.org/p/tryserver.chromium/builders/linux_chromium_chrome...)

[commit-bot: I haz the power, 2014-05-16 06:11:54]

The CQ bit was unchecked by commit-bot@chromium.org

[commit-bot: I haz the power, 2014-05-16 06:11:55]

Failed to apply patch for @master/chrome/chrome_installer.gypi:
While running svn add @master --force --config-dir
/b/infra_internal/commit_queue/subversion_config --non-interactive;
  svn: '@master' is just a peg revision. Maybe try '@master@' instead?

Patch:       @master/chrome/chrome_installer.gypi
Index: chrome/chrome_installer.gypi
diff --git @master/chrome/chrome_installer.gypi
@master/chrome/chrome_installer.gypi
index
b25aa4abd97bf8aac1f0f281e36ec460f137f771..a2dabf6d48b91734f8da1055e9b091fc171b4b4e
100644
--- a/@master/chrome/chrome_installer.gypi
+++ b/@master/chrome/chrome_installer.gypi
@@ -295,9 +295,6 @@
             'installer/setup/uninstall.h',
           ],
           'msvs_settings': {
-            'VCCLCompilerTool': {
-              'EnableEnhancedInstructionSet': '4',  # NoExtensions
-            },
             'VCLinkerTool': {
               'SubSystem': '2',     # Set /SUBSYSTEM:WINDOWS
             },
@@ -357,6 +354,13 @@
                  'branding_dir_100': 'app/theme/default_100_percent/chromium',
               },
             }],
+            ['target_arch=="ia32"', {
+              'msvs_settings': {
+                'VCCLCompilerTool': {
+                  'EnableEnhancedInstructionSet': '4',  # NoExtensions
+                },
+              },
+            }],
           ],
         },
         {

[Will Harris, 2014-05-16 16:30:03]

The CQ bit was checked by wfh@chromium.org

[commit-bot: I haz the power, 2014-05-16 16:30:34]

CQ is trying da patch. Follow status at
 https://chromium-status.appspot.com/cq/wfh@chromium.org/288353004/20001

[commit-bot: I haz the power, 2014-05-16 16:31:49]

Change committed as 271031