[PATCH] D58933: [opt] Report if the provided architecture is invalid.

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 4 15:55:10 PST 2019


efriedma accepted this revision.
efriedma added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: llvm/tools/opt/opt.cpp:644
+             ModuleTriple.getArchName() != "") {
+    errs() << argv[0] << ": invalid architecture '"
+           << ModuleTriple.getArchName() << "' provided.\n";
----------------
I would probably say "unrecognized architecture" or something like that... but that's just nitpicking.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D58933





More information about the llvm-commits mailing list