[PATCH] D60572: creduce-clang-crash: Use `!` instead of `not`

2019-04-11 Thread Amy Huang via Phabricator via cfe-commits
akhuang added a comment. Looks good- if we remove `not` there are a few other things that should be removed Comment at: clang/utils/creduce-clang-crash.py:138 if "fatal error:" in msg_re: self.is_crash = False break also wouldn't

[PATCH] D60572: creduce-clang-crash: Use `!` instead of `not`

2019-04-11 Thread Nico Weber via Phabricator via cfe-commits
thakis abandoned this revision. thakis added a comment. This seems to break the script, nevermind! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D60572/new/ https://reviews.llvm.org/D60572 ___ cfe-commits mailing list

[PATCH] D60572: creduce-clang-crash: Use `!` instead of `not`

2019-04-11 Thread Nico Weber via Phabricator via cfe-commits
thakis created this revision. thakis added a reviewer: akhuang. The run script runs in bash, so no need to depend on llvm's `not` binary. https://reviews.llvm.org/D60572 Files: clang/utils/creduce-clang-crash.py Index: clang/utils/creduce-clang-crash.py