[clang] [llvm] Call setup_host_tools for more tools. (PR #118635)

James Henderson via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 4 05:53:05 PST 2024


jh7370 wrote:

> This has no effect in LLVM itself, which does not make use of this, but is going to be used in DPC++'s libsycl. That is meant to be upstreamed at some point, so it probably makes sense to get this into LLVM already and avoid future conflicts.

I don't know what others would think about this, but without the subsequent changes that make use of this, I don't think this should be landed in LLVM. After all, plans might change, the community might refuse the subsequent PRs etc, resulting in dead code lying around.

https://github.com/llvm/llvm-project/pull/118635


More information about the cfe-commits mailing list