[PATCH] D64572: [UpdateTestChecks] Emit warning when invalid test paths

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 11 09:45:00 PDT 2019


xbolva00 added a comment.

In D64572#1580897 <https://reviews.llvm.org/D64572#1580897>, @jdoerfert wrote:

> LGTM.


Do you think hard error is better? I switched to warning since if somebody use

./update_test_checks.py fileA fileB fileC

and fileC does not exists, the script will exit.

New behaviour, we process fileA and fileB and we emit warnign for fileC.


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

https://reviews.llvm.org/D64572





More information about the llvm-commits mailing list