clang/win: Start work on getting clang/win working in gn.

build/config/compiler/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/android/config.gni")
 import("//build/config/chrome_build.gni")
 if (current_cpu == "arm") {
   import("//build/config/arm.gni")
 }
 if (current_cpu == "mipsel" || current_cpu == "mips64el") {
@@ skipping 84 matching lines @@
   # GCC flags, and then we handle the other non-Windows platforms specifically
   # below.
   if (is_win) {
     # Windows compiler flags setup.
     # -----------------------------
     cflags += [
       "/Gy",  # Enable function-level linking.
       "/GS",  # Enable buffer security checking.
       "/FS",  # Preserve previous PDB behavior.
     ]
+
+    # Building with Clang on Windows is a work in progress and very
+    # experimental. See crbug.com/82385.
+    # Keep this in sync with the similar block in build/common.gypi
+    if (is_clang) {
+      cflags += [
+        "-fmsc-version=1800",
+
+        # Many files use intrinsics without including this header.
+        # TODO(hans): Fix those files, or move this to sub-GYPs.
+        "/FIIntrin.h",
+      ]
+      if (current_cpu == "x32") {
+        cflags += [ "/fallback" ]
+      }
+    }
   } else {
     # Common GCC compiler flags setup.
     # --------------------------------
     cflags += [ "-fno-strict-aliasing" ]  # See http://crbug.com/32204
     cflags_cc += [
       "-fno-threadsafe-statics",
 
       # Not exporting C++ inline functions can generally be applied anywhere
       # so we do so here. Normal function visibility is controlled by
       # //build/config/gcc:symbol_visibility_hidden.
@@ skipping 651 matching lines @@
   }
 }
 
 # Warnings ---------------------------------------------------------------------
 #
 # This is where we disable various warnings that we've decided aren't
 # worthwhile, and enable special warnings.
 
 config("default_warnings") {
   if (is_win) {
-    cflags = [
-      "/WX",  # Treat warnings as errors.
+    cflags = []
 
+    if (!is_clang || current_cpu != "x32") {
+      cflags += [ "/WX" ]  # Treat warnings as errors.
+    }
+
+    cflags += [
       # Warnings permanently disabled:
 
       # TODO(GYP) The GYP build doesn't have this globally enabled but disabled
       # for a bunch of individual targets. Re-enable this globally when those
       # targets are fixed.
       "/wd4018",  # Comparing signed and unsigned values.
 
       # C4127: conditional expression is constant
       # This warning can in theory catch dead code and other problems, but
       # triggers in far too many desirable cases where the conditional
@@ skipping 55 matching lines @@
       "/wd4996",  # Deprecated function warning.
     ]
 
     # VS xtree header file needs to be patched or 4702 (unreachable code
     # warning) is reported if _HAS_EXCEPTIONS=0. Disable the warning if xtree is
     # not patched.
     if (!msvs_xtree_patched &&
         exec_script("../../win_is_xtree_patched.py", [], "value") == 0) {
       cflags += [ "/wd4702" ]  # Unreachable code.
     }
+
+    # Building with Clang on Windows is a work in progress and very
+    # experimental. See crbug.com/82385.
+    # Keep this in sync with the similar block in build/common.gypi
+    if (is_clang) {
+      cflags += [
+        # TODO(hans): Make this list shorter eventually.
+        "-Qunused-arguments",
+        "-Wno-c++11-compat-deprecated-writable-strings",
+        "-Wno-deprecated-declarations",
+        "-Wno-empty-body",
+        "-Wno-enum-conversion",
+        "-Wno-extra-tokens",
+        "-Wno-ignored-attributes",
+        "-Wno-incompatible-pointer-types",
+        "-Wno-int-to-void-pointer-cast",
+        "-Wno-invalid-noreturn",
+        "-Wno-logical-op-parentheses",
+        "-Wno-microsoft",
+        "-Wno-missing-braces",
+        "-Wno-missing-declarations",
+        "-Wno-msvc-include",
+        "-Wno-null-dereference",
+        "-Wno-overloaded-virtual",
+        "-Wno-parentheses",
+        "-Wno-pointer-sign",
+        "-Wno-reorder",
+        "-Wno-return-type-c-linkage",
+        "-Wno-self-assign",
+        "-Wno-sometimes-uninitialized",
+        "-Wno-switch",
+        "-Wno-tautological-compare",
+        "-Wno-unknown-pragmas",
+        "-Wno-unsequenced",
+        "-Wno-unused-function",
+        "-Wno-unused-private-field",
+        "-Wno-unused-value",
+        "-Wno-unused-variable",
+        "-Wno-unused-local-typedef",  # http://crbug.com/411648
+        "-Wno-inconsistent-missing-override",  #http://crbug.com/428099

[Nico, 2015/06/02 22:52:58]

`gn format` insisted that I don't put a space after # here

[scottmg, 2015/06/02 23:02:21]

o_O sorry, not sure why
https://code.google.com/p/chromium/issues/detail?id=495862.

+      ]
+    }
   } else {
     # Common GCC warning setup.
     cflags = [
       # Enables.
       "-Wendif-labels",  # Weird old-style text after an #endif.
       "-Werror",  # Warnings as errors.
 
       # Disables.
       "-Wno-missing-field-initializers",  # "struct foo f = {0};"
       "-Wno-unused-parameter",  # Unused function parameters.
     ]
     cflags_cc = []
 
     if (is_mac) {
       cflags += [ "-Wnewline-eof" ]
       if (!is_nacl) {
         # When compiling Objective-C, warns if a method is used whose
         # availability is newer than the deployment target. This is not
         # required when compiling Chrome for iOS.
         cflags += [ "-Wpartial-availability" ]
       }
     }
 
-    if (is_clang) {
-      cflags += [
-        # This warns on using ints as initializers for floats in
-        # initializer lists (e.g. |int a = f(); CGSize s = { a, a };|),
-        # which happens in several places in chrome code. Not sure if
-        # this is worth fixing.
-        "-Wno-c++11-narrowing",
-
-        # Don't die on dtoa code that uses a char as an array index.
-        # This is required solely for base/third_party/dmg_fp/dtoa.cc.
-        # TODO(brettw) move this to that project then!
-        "-Wno-char-subscripts",
-
-        # Warns on switches on enums that cover all enum values but
-        # also contain a default: branch. Chrome is full of that.
-        "-Wno-covered-switch-default",
-
-        # Clang considers the `register` keyword as deprecated, but e.g.
-        # code generated by flex (used in angle) contains that keyword.
-        # http://crbug.com/255186
-        "-Wno-deprecated-register",
-
-        # TODO(thakis): This used to be implied by -Wno-unused-function,
-        # which we no longer use. Check if it makes sense to remove
-        # this as well. http://crbug.com/316352
-        "-Wno-unneeded-internal-declaration",
-
-        # TODO(thakis): Remove, http://crbug.com/263960
-        "-Wno-reserved-user-defined-literal",
-      ]
-
-      # NaCl's Clang compiler and Chrome's hermetic Clang compiler will almost
-      # always have different versions. Certain flags may not be recognized by
-      # one version or the other.
-      if (!is_nacl) {
-        # Flags NaCl does not recognize.
-        cflags += [
-          # TODO(hans): Get this cleaned up.
-          "-Wno-inconsistent-missing-override",
-        ]
-      }
-    }
     if (gcc_version >= 48) {
       cflags_cc += [
         # See comment for -Wno-c++11-narrowing.
         "-Wno-narrowing",
 
         # TODO(thakis): Remove, http://crbug.com/263960
         "-Wno-literal-suffix",
       ]
     }
 
@@ skipping 25 matching lines @@
       ]
     }
 
     if (gcc_version >= 48) {
       # Don't warn about the "typedef 'foo' locally defined but not used"
       # for gcc 4.8.
       # TODO: remove this flag once all builds work. See crbug.com/227506
       cflags += [ "-Wno-unused-local-typedefs" ]
     }
   }
+
+  if (is_clang) {
+    cflags += [
+      # This warns on using ints as initializers for floats in
+      # initializer lists (e.g. |int a = f(); CGSize s = { a, a };|),
+      # which happens in several places in chrome code. Not sure if
+      # this is worth fixing.
+      "-Wno-c++11-narrowing",
+
+      # Don't die on dtoa code that uses a char as an array index.
+      # This is required solely for base/third_party/dmg_fp/dtoa.cc.
+      # TODO(brettw) move this to that project then!
+      "-Wno-char-subscripts",
+
+      # Warns on switches on enums that cover all enum values but
+      # also contain a default: branch. Chrome is full of that.
+      "-Wno-covered-switch-default",
+
+      # Clang considers the `register` keyword as deprecated, but e.g.
+      # code generated by flex (used in angle) contains that keyword.
+      # http://crbug.com/255186
+      "-Wno-deprecated-register",
+
+      # TODO(thakis): This used to be implied by -Wno-unused-function,
+      # which we no longer use. Check if it makes sense to remove
+      # this as well. http://crbug.com/316352
+      "-Wno-unneeded-internal-declaration",
+
+      # TODO(thakis): Remove, http://crbug.com/263960
+      "-Wno-reserved-user-defined-literal",
+    ]
+
+    # NaCl's Clang compiler and Chrome's hermetic Clang compiler will almost
+    # always have different versions. Certain flags may not be recognized by
+    # one version or the other.
+    if (!is_nacl) {
+      # Flags NaCl does not recognize.
+      cflags += [
+        # TODO(hans): Get this cleaned up.
+        "-Wno-inconsistent-missing-override",
+      ]
+    }
+  }
 }
 
 # This will generate warnings when using Clang if code generates exit-time
 # destructors, which will slow down closing the program.
 # TODO(thakis): Make this a blacklist instead, http://crbug.com/101600
 config("wexit_time_destructors") {
-  if (is_clang) {
+  # TODO: Enable on Windows too, http://crbug.com/404525
+  if (is_clang && !is_win) {
     cflags = [ "-Wexit-time-destructors" ]
   }
 }
 
 # On Windows compiling on x64, VC will issue a warning when converting
 # size_t to int because it will truncate the value. Our code should not have
 # these warnings and one should use a static_cast or a checked_cast for the
 # conversion depending on the case. However, a lot of code still needs to be
 # fixed. Apply this config to such targets to disable the warning.
 #
@@ skipping 184 matching lines @@
       cflags += [ "-gsplit-dwarf" ]
     }
   }
 }
 
 config("no_symbols") {
   if (!is_win) {
     cflags = [ "-g0" ]
   }
 }