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

Sat Jan 18 06:48:14 PST 2020

epastor added a comment.

In D72679#1828049 <https://reviews.llvm.org/D72679#1828049>, @RKSimon wrote:

> @epastor This is causing build bots failures, please can you take a look?  http://lab.llvm.org:8011/builders/llvm-clang-win-x-armv7l/builds/3121

Looks real to me - this entire tool is premised on the availability of the (i386|x86_64)-pc-windows targets, since it's intended to imitate ml.exe & ml64.exe. Apparently at least one isn't available when building in this environment? Any reason for that?

Also, would there be a way to keep this tool from building (and exclude it from tests) if those targets aren't available?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72679