[llvm] Use exec_script_allowlist in //llvm/utils/gn/.gn (PR #134172)
Nico Weber via llvm-commits
llvm-commits at lists.llvm.org
Mon Apr 7 10:17:43 PDT 2025
https://github.com/nico approved this pull request.
lg, but please give me a bit to update the bots.
Do you want to update the comment right above too?
Could you also update https://github.com/llvm/llvm-project/blob/783201b184572a07efe2dc6b6b9110873421cf11/llvm/utils/gn/build/toolchain/BUILD.gn#L86 ? (Move it to top of file; update comment.)
Another idea could be to assign to one or the other depending on `gn_version`.
But I think fairly few people use this build, and it's fairly easy to update GN (just run `llvm/utils/gn/get.py`), so I think it's fine to just require a new(ish) version of GN.
https://github.com/llvm/llvm-project/pull/134172
More information about the llvm-commits
mailing list