jdoerfert reopened this revision.
jdoerfert added a comment.
This revision is now accepted and ready to land.

The test coverage here is not acceptable:

1. The command line of the tests is far from what it should be (see other 
tests).
2. The check lines do little to prevent regressions in the future.
3. There are no test for wrong usage, errors, ...


Repository:
  rL LLVM

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

https://reviews.llvm.org/D61809



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to