[PATCH] D104008: [ms] [llvm-ml] Warn on command-line redefinition

[Nico Weber via Phabricator via llvm-commits]

thakis added inline comments.


================
Comment at: llvm/test/tools/llvm-ml/command_line_defines.asm:1
-; RUN: llvm-ml -filetype=s %s /Fo - /DT1=test1 /D T2=test2 | FileCheck %s
+; RUN: llvm-ml -filetype=s %s /Fo - /DT1=test1 /D T2=test2 /Dtest5=def | FileCheck %s
 
----------------
thakis wrote:
> Maybe you need `2>&1` in front of the `|` so that lit sees stderr?
Wait you meant the other test. One moment…


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104008