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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits