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

[James Henderson via Phabricator via llvm-commits]

jhenderson added a comment.

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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96371