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

Reply via email to