Fznamznon added inline comments.
================ Comment at: clang/test/Driver/x-args.c:12 +// RUN: not %clang_cl /TC /WX /clang:-xc /clang:-E /clang:-dM %s 2>&1 | FileCheck -check-prefix=CL %s +// CL-NOT: '-x c' after last input file has no effect +// CL: error: unsupported option '-x c'; did you mean '/TC' or '/TP'? ---------------- Fznamznon wrote: > MaskRay wrote: > > You can remove this `CL-NOT` negative pattern. Instead, use > > `--check-prefix=CL --implicit-check-not=error:` to check that there is no > > other error. > I probably don't fully understand the suggestion, but there is a `error:` > that is being checked in this test. The one that this revision adds. It is > checked on the next line. Is that ok if I submit the patch "as is"? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D142757/new/ https://reviews.llvm.org/D142757 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits