================
@@ -525,6 +525,19 @@ TEST(CommandMangler, RespectsOriginalSysroot) {
Not(HasSubstr(testPath("fake/sysroot"))));
}
}
+TEST(CommandMangler, ClangUnknownArgs) {
+ // Check that clang-cl-specific will drop unknown flags
+ const auto Mangler = CommandMangler::forTests();
+ {
+ tooling::CompileCommand Cmd;
+ Cmd.CommandLine = {"clang++", "-std=c++23", "--unknown-flag=abcd.flag"};
+ Mangler(Cmd, "foo.cc");
+ EXPECT_THAT(llvm::join(Cmd.CommandLine, " "),
+ Not(HasSubstr("--unknown-flag=abcd.flag")));
+ EXPECT_THAT(llvm::join(Cmd.CommandLine, " "),
+ HasSubstr("--std=c++23"));
----------------
Decodetalkers wrote:
ok, that was a mistake, I have fixed that
https://github.com/llvm/llvm-project/pull/200001
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
