[PATCH] D139167: [clang][Windows]Ignore Options '/d1nodatetime' and '/d1import_no_registry'

Hans Wennborg via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Dec 2 04:00:34 PST 2022


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



More information about the cfe-commits mailing list