dexonsmith accepted this revision.
dexonsmith added a comment.

LGTM with a minor comment on the test.



================
Comment at: clang/test/Driver/driver-only.c:13
+
+// Check that -fdriver-only respects -v.
+//
----------------
Should there be a `-check-epmty` or similar when `-v` is not passed? (This 
tests the positive, but I don’t see a test that it’s not ALWAYS verbose.)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D127408

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

Reply via email to