[PATCH] D58614: Fixed a typo in the test s/CEHCK/CHECK/
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 25 07:29:25 PST 2019
lebedev.ri added a comment.
In D58614#1408924 <https://reviews.llvm.org/D58614#1408924>, @jsji wrote:
> LGTM. Thanks for catching this, I am surprised that those typos slipped through ... FYI. @HLJ2009
Two things here.
1. Why is `CHECK-NEXT` not being used?
2. Why is `./utils/update_llc_test_checks.py` not being used? That dramatically reduces chances of things like this, and spares one from actually having to manually write proper check lines. (thus improves test coverage. one will of course still need to verify that the checks are correct.)
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D58614/new/
https://reviews.llvm.org/D58614
More information about the llvm-commits
mailing list