Skip to content

Commit 310b0a1

Browse files
authored
[llvm-ml] Adds /quiet flag to llvm-ml (llvm#107308)
In PR llvm#106794, it was noted that `llvm-ml` does not support the `/quiet` flag. The original reason it was added by Microsoft to `ml`/`ml64` was to remove extraneous CMake build output (see [CMake GitLab issue](https://gitlab.kitware.com/cmake/cmake/-/issues/23537)) much like in the linked PR . If the goal is for `llvm-ml` to be a drop-in replacement for `ml`/`ml64`, then I think it makes sense to support the `/quiet` flag, much like how `/nologo` is supported.
1 parent 861caf9 commit 310b0a1

File tree

2 files changed

+2
-0
lines changed

2 files changed

+2
-0
lines changed

llvm/test/tools/llvm-ml/run.asm

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
; RUN: llvm-ml --help | FileCheck %s
2+
; RUN: llvm-ml /nologo /quiet
23

34
; CHECK: USAGE: llvm-ml

llvm/tools/llvm-ml/Opts.td

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,7 @@ def define : MLJoinedOrSeparate<"D">, MetaVarName<"<macro>=<value>">,
7272
HelpText<"Define <macro> to <value> (or blank if <value> "
7373
"omitted)">;
7474
def no_logo : MLFlag<"nologo">, HelpText<"">;
75+
def quiet : MLFlag<"quiet">, HelpText<"">;
7576
def output_file : MLJoinedOrSeparate<"Fo">, HelpText<"Names the output file">;
7677
def include_path : MLJoinedOrSeparate<"I">,
7778
HelpText<"Sets path for include files">;

0 commit comments

Comments
 (0)