[PATCH] D96371: [lit] Add --ignore-fail

[Thomas Preud'homme via Phabricator via llvm-commits]

thopre added a comment.

In D96371#2553434 <https://reviews.llvm.org/D96371#2553434>, @jhenderson wrote:

> Personally, I think it should be the default to run all testsuites rather than exiting after the first failure. The exit code should then be the combined one, i.e. whether any of the suites has failed.
>
> Is it not possible to pass the exit code up the stack rather than `sys.exit(1)` there? The caller (llvm-lit.py?) of the multiple suites would then be responsible for checking all `main()` calls passed.

Was about to comment the same. Something a la `--keep-going` instead of `--ignore-fail`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D96371/new/

https://reviews.llvm.org/D96371