[PATCH] D112399: Get Bazel building `//clang` on Windows with clang-cl.

Chandler Carruth via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 1 19:55:07 PDT 2021


chandlerc added inline comments.


================
Comment at: utils/bazel/llvm-project-overlay/clang/BUILD.bazel:1828
     ],
-    copts = [
-        "-Wno-uninitialized",
-    ],
+    copts = select({
+        "@bazel_tools//src/conditions:windows": [],
----------------
rnk wrote:
> Enabling warnings is good, but what made this change necessary? The `-Wno-` flag should have worked with clang-cl. If this change isn't necessary, maybe go ahead and delete this copt in a separate commit.
Restored.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112399/new/

https://reviews.llvm.org/D112399



More information about the cfe-commits mailing list