[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