[PATCH] D56789: Allow CHECK-SAME, NEXT and EMPTY after CHECK-DAG

Paul Robinson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 23 11:44:28 PST 2019


probinson added a comment.

The revision title and summary contradict each other. The title says to allow something, but the summary says that already works; then the summary says CHECK-DAG can't be first, while the `check-dag.txt` test clearly already has CHECK-DAG as the first directive.
Please fix up the title and summary so that they correctly describe what you are trying to accomplish. This will help reviewers to make sure the patch does what you want.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D56789





More information about the llvm-commits mailing list