[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