dexonsmith requested changes to this revision. dexonsmith added inline comments. This revision now requires changes to proceed.
================ Comment at: clang/test/ClangScanDeps/Inputs/removed-args/cdb.json.template:4 + "directory": "DIR", + "command": "clang -fsyntax-only DIR/tu.c -fmodules -fimplicit-module-maps -fmodules-cache-path=DIR/cache -include DIR/header.h -o DIR/tu.o", + "file": "DIR/tu.c" ---------------- It looks to me like that this test would have passed before the code change, since there was already logic for dropping `-include`. Can you also add some other argument(s) that wouldn't have been dropped before, but will be now? ================ Comment at: clang/test/ClangScanDeps/removed-args.c:3 +// RUN: cp %S/Inputs/removed-args/* %t + +// RUN: sed "s|DIR|%/t|g" %S/Inputs/removed-args/cdb.json.template > %t/cdb.json ---------------- Please add a comment here explaining what args are expected to be dropped and why, to help future readers to understand the testcase. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108710/new/ https://reviews.llvm.org/D108710 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
