Make Win Clang ASan builds work

build/config/BUILD.gn

 # Copyright (c) 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("//build/config/allocator.gni")
 import("//build/config/chrome_build.gni")
 import("//build/config/chromecast_build.gni")
 import("//build/config/crypto.gni")
 import("//build/config/dcheck_always_on.gni")
 import("//build/config/features.gni")
@@ skipping 390 matching lines @@
   }
 
   # If we're using the prebuilt instrumented libraries with the sanitizers, we
   # need to add ldflags to every binary to make sure they are picked up.
   if (prebuilt_instrumented_libraries_available) {
     configs += [ "//third_party/instrumented_libraries:prebuilt_ldflags" ]
   }
   if (use_locally_built_instrumented_libraries) {
     configs += [ "//third_party/instrumented_libraries:locally_built_ldflags" ]
   }
+  configs += [ "//build/config/sanitizers:executable_config" ]
 }
 
 # Shared library configs -------------------------------------------------------
 
 # This config defines the configs applied to all shared libraries.
 config("shared_library_config") {
   configs = []
 
   if (is_win) {
     configs += _windows_linker_configs
   } else if (is_mac) {
     configs += [ "//build/config/mac:mac_dynamic_flags" ]
   } else if (is_ios) {
     configs += [ "//build/config/ios:ios_dynamic_flags" ]
   } else if (is_chromecast) {
     configs += [ "//build/config/chromecast:shared_library_config" ]
   }
 
   # If we're using the prebuilt instrumented libraries with the sanitizers, we
   # need to add ldflags to every binary to make sure they are picked up.
   if (prebuilt_instrumented_libraries_available) {
     configs += [ "//third_party/instrumented_libraries:prebuilt_ldflags" ]
   }
   if (use_locally_built_instrumented_libraries) {
     configs += [ "//third_party/instrumented_libraries:locally_built_ldflags" ]
   }
+  configs += [ "//build/config/sanitizers:executable_config" ]

[Nico, 2016/08/02 14:20:51]

It seems very surprising to me that shared_library_config here depends on
executable_config in the other place. I think it's better to have a
shared_library_config in the other place as well.

[Dirk Pranke, 2016/08/02 17:19:03]

Agreed; This patch was based on etienneb's patch, and I hadn't noticed this
line, but it should change.

[etienneb, 2016/08/02 22:50:26]

I agree too.
This was a quick try to make the build work.

 }
 
 # Add this config to your target to enable precompiled headers.
 #
 # Precompiled headers are done on a per-target basis. If you have just a couple
 # of files, the time it takes to precompile (~2 seconds) can actually be longer
 # than the time saved. On a Z620, a 100 file target compiles about 2 seconds
 # faster with precompiled headers, with greater savings for larger targets.
 #
 # Recommend precompiled headers for targets with more than 50 .cc files.
@@ skipping 22 matching lines @@
       # file doesn't exist, no error will be generated (probably MS tested this
       # case but forgot the other one?). To reproduce this error, do a build,
       # then delete the precompile.c.obj file, then build again.
       cflags_c = [ "/wd4206" ]
     } else if (is_mac) {
       precompiled_header = "build/precompile.h"
       precompiled_source = "//build/precompile.h"
     }
   }
 }