hans added a comment.

I hadn't seen `/d1` flags before. I suppose they relate to preprocessor stuff 
as opposed to `/d2` flags.

I think the "Unsupported" section below may be better for these. In that case 
clang-cl will parse the option and warn that it's ignoring it. On line 6931 
there's a catch-all for all flags starting with "d2". Perhaps we should add one 
for "d1" too?

Also, please add these to the "Unsupported but parsed options" tests in 
clang/test/Driver/cl-options.c


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D139167/new/

https://reviews.llvm.org/D139167

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to