[llvm] [CI] Disable flang from pre-commit tests when Flang files are not touched (PR #93485)

Valentin Clement バレンタイン クレメン via llvm-commits llvm-commits at lists.llvm.org
Wed May 29 13:11:10 PDT 2024


clementval wrote:

> > > Isn't is tested in post-commit bots?
> > 
> > 
> > Yes, I just don't see the problem to have it in linux pre-commit since it was reliable so far until now. If it's a problem of build time or load then it's a different issue.
> 
> If there's a way to run on Linux but not Windows, then I don't mind. I just want to make sure we only add to pre-commit what we know is stable, but letting it run post-commit for a few months without crashes.

Yeah that makes sense to enable pre-commit that are stable only. If we can keep linux that would be great. Even with pre-commit enabled it happened a couple of times that some patches were committed even with obvious failure in the pre commit CI.

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


More information about the llvm-commits mailing list