[PATCH] D143295: [bazel] Move bazel configuration to a Python script

NAKAMURA Takumi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 12 03:35:13 PST 2023


chapuni added a comment.

I supposed `llvm_configure` is the only public entry for users.
I won't oppose if `overlay_directories.py` may be assumed as 2nd public entry.

One concern. It seems this change tends to miss update of `llvm/CMakeLists.txt`.
(for example, `bazel build` doesn't detect changes if `llvm/CMakeLists.txt` is changed.)
That said, `utils/bazel` has been flaky for change of directory structure to be mirrored.
We could run `bazel sync` to satisfy changes, I think


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143295



More information about the llvm-commits mailing list