[PATCH] D48702: objdump: Better handling of Mach-O universal binaries

[Dave Lee via Phabricator via llvm-commits]

kastiglione added inline comments.


================
Comment at: tools/llvm-objdump/MachODump.cpp:1961
+void llvm::ParseInputMachO(StringRef Filename) {
+  if (!ValidateArchFlags())
+    return;
----------------
compnerd wrote:
> This leaves the old behavior in place, but I think it would be nicer if the `arch` flag was validated earlier.
Did you have something specific in mind? By the caller, before calling this function? Or something else…


Repository:
  rL LLVM

https://reviews.llvm.org/D48702