[PATCH] D90061: [ms] [llvm-ml] Allow the /Zs parameter as a synonym for -filetype=null

[Nico Weber via Phabricator via llvm-commits]

thakis requested changes to this revision.
thakis added a comment.
This revision now requires changes to proceed.

Still no test, as far as I can tell. I think the suggestion below turns this into a 3-line diff (+ test).



================
Comment at: llvm/tools/llvm-ml/Opts.td:48
                              "the .asm extension">;
+def parse_only : MLFlag<"Zs">, HelpText<"Run a syntax-check only">;
 
----------------
Instead of all the DerivedArgList changes below, doesn't just saying `..., Alias<filetype>, AliasArgs<"none">` here do the same thing?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90061