[PATCH] D72679: [ms] [llvm-ml] Add placeholder for llvm-ml, based on llvm-mc

[Eric Astor via Phabricator via llvm-commits]

epastor added a comment.

In D72679#1821730 <https://reviews.llvm.org/D72679#1821730>, @thakis wrote:

> It might be nice to add a test that just runs `llvm-ml --help` to check that the binary runs.


Done.

> I'd imagine that once this is mostly done, we want to busy-box into clang-cl so that `clang-cl file.asm` runs the ml-style asm parser by default, and we'd have some -fasm-mode flag to override this (so that .s files can be parsed in ml mode and .asm files in gas mode, if desired). So far, this is all compatible with that, but something to keep in mind going forward.

Something like that, once this is mostly done... though I'm not familiar with what you mean by "busy-box into clang-cl". I don't think that should be too tricky.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72679