[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