[clang] [llvm] [Clang] Add env var for nvptx-arch/amdgpu-arch timeout (PR #102521)
Joseph Huber via cfe-commits
cfe-commits at lists.llvm.org
Fri Aug 9 08:56:34 PDT 2024
jhuber6 wrote:
> > Possibly worth increasing it to a minute then,
>
> Should we also land the env var? If so, I'll just extend this patch.
>
> Either way, any objection to moving `[Default]SecondsToWait` into executeToolChainProgram so we're maintaining it in just one place?
>
> > this is basically just a last ditch effort to prevent builds from hanging forever. I've had it happen personally where either the CUDA or AMDGPU drivers were stuck, and when I tried to rebuild `libc` `ninja` just hung indefinitely and it took me awhile to figure out why.
>
> I'm not sure how often the situation you describe occurs in practice. Hopefully a 60-second pause on a simple compilation doesn't prompt the same confusion and a premature Ctrl+C.
You can just do both in the same patch, I don't think it's too important.
https://github.com/llvm/llvm-project/pull/102521
More information about the cfe-commits
mailing list