4vtomat added inline comments.
================ Comment at: clang/test/Driver/print-supported-extensions.c:6 +// RUN: %clang --target=riscv64 --print-supported-extensions 2>&1 | \ +// RUN: FileCheck %s --check-prefix=CHECK-RISCV + ---------------- MaskRay wrote: > MaskRay wrote: > > Use `--implicit-check-not=warning:` instead of `// CHECK-NOT: warning: > > argument unused during compilation`. > > > > We additionally check that there is no warning and the warning doesn't come > > after the output from `--print-supported-extensions`. > For a file with just one prefix, conventionally we just use the default > `CHECK` and omit `--check-prefix`. > > Have you ever checked that the test detects issues? `CHECK` below are ignored > with `--check-prefix=CHECK-RISCV`. Oh, I missed it, CHECK-NEXT are all ignored... thanks!! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D146054/new/ https://reviews.llvm.org/D146054 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
