jhuber6 added inline comments.
================ Comment at: clang/test/Driver/unknown-arg.c:74 + +// RUN: %clang -### --offload-arch=sm_70 -o ffload-device-only -Werror=unknown-argument %s ---------------- tra wrote: > jhuber6 wrote: > > tra wrote: > > > Has this patch been updated with incomplete changes? This RUN line seems > > > to miss `2>&1 | FileCheck` and the check lines. > > I'm relying on the fact that `-Werror` will cause clang to return a > > non-zero error code and make the test fail if any warnings show up. > Got it. I've missed that. You were somewhat right, I had `-###` which apparently makes clang return 0 even on an error. So thanks for making me double check. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D135389/new/ https://reviews.llvm.org/D135389 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
