V8 Buildbot: Disable lsan on bots where it's not supported.

scripts/slave/recipe_modules/chromium/config.py

 # Copyright 2013 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.
 
 import pipes
 
 from recipe_engine.config import config_item_context, ConfigGroup
 from recipe_engine.config import Dict, List, Single, Static, Set, BadConf
 from recipe_engine.config_types import Path
 
@@ skipping 375 matching lines @@
 @config_ctx(deps=['compiler'])
 def asan(c):
   if 'clang' not in c.compile_py.compiler:  # pragma: no cover
     raise BadConf('asan requires clang')
   c.runtests.swarming_tags |= {'asan:1'}
   if c.TARGET_PLATFORM in ['mac', 'win']:
     # Set fastbuild=0 and prevent other configs from changing it.
     fastbuild(c, invert=True, optional=False)
 
   c.gyp_env.GYP_DEFINES['asan'] = 1
-  if c.TARGET_PLATFORM != 'android':
-    # LSAN isn't supported on Android.
+  if c.TARGET_PLATFORM != 'android' and c.TARGET_BITS == 64:
+    # LSAN isn't supported on Android or 32 bits platforms.
     c.gyp_env.GYP_DEFINES['lsan'] = 1
 
 @config_ctx(deps=['compiler'])
 def lsan(c):
   c.runtests.enable_lsan = True
   c.runtests.swarming_extra_args += ['--lsan=1']
   c.runtests.swarming_tags |= {'lsan:1'}
 
 # TODO(infra,earthdok,glider): Make this a gyp variable. This is also not a
 # good name as only v8 builds release symbolized with -O2 while
@@ skipping 101 matching lines @@
 
 @config_ctx(includes=['chromium_win_clang'])
 def chromium_win_clang_tot(c):
   c.env.LLVM_FORCE_HEAD_REVISION = 'YES'
 
 @config_ctx(includes=['chromium_win_clang', 'official'])
 def chromium_win_clang_official(c):
   pass
 
 @config_ctx(includes=['chromium_win_clang', 'asan', 'static_library'])
 def chromium_win_clang_asan(c):

[Michael Achenbach, 2015/07/15 07:16:37]

This is now covered by excluding 32 bits above.

-  # Clear lsan configuration for win.
-  del c.gyp_env.GYP_DEFINES['lsan']
   # These are set on the lkgr bot, and the fyi bots should match the lkgr bot.
   # TODO(thakis): Once the lkgr bot uses recipes, the lkgr and the fyi bots
   # should use the same context to ensure they use the same gyp defines.
   c.gyp_env.GYP_DEFINES['enable_ipc_fuzzer'] = 1
   c.gyp_env.GYP_DEFINES['v8_enable_verify_heap'] = 1
 
 @config_ctx(includes=['chromium_win_clang_asan'])
 def chromium_win_clang_asan_tot(c):
   c.env.LLVM_FORCE_HEAD_REVISION = 'YES'
 
@@ skipping 253 matching lines @@
 def chromium_deterministic_build(c):
   c.gyp_env.GYP_DEFINES['dont_embed_build_metadata'] = 1
 
 @config_ctx()
 def chromium_mac_sdk_10_10(c):
   c.gyp_env.GYP_DEFINES['mac_sdk_min='] = '10.10'
 
 @config_ctx(includes=['chromium_clang'])
 def cast_linux(c):
   c.gyp_env.GYP_DEFINES['chromecast'] = 1