[PATCH] D156433: [bazel] Include builtin headers with clang-tidy

Keith Smiley via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 27 09:33:31 PDT 2023


keith added inline comments.


================
Comment at: utils/bazel/llvm-project-overlay/clang-tools-extra/defs.bzl:7
+
+def _symlink_impl(ctx):
+    copied_files = []
----------------
I'm not sure if the LLVM repo wants this but there is a rule in the bazel-skylib for this https://github.com/bazelbuild/bazel-skylib/blob/main/docs/copy_file_doc.md#copy_file-allow_symlink


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D156433



More information about the llvm-commits mailing list