[PATCH] D27360: [clang] Fix D26214: Move error handling out of MC and to the callers.

[Saleem Abdulrasool via Phabricator via llvm-commits]

compnerd added a comment.

I think that the argument parsing should really be hoisted into the driver rather than pushed down into the assembler.  The driver can parse and validate the options before passing them down, so when cc1as gets it, it will simply set the value.

The newly introduced diagnostics, especially right now, are not driver related.  Nor are they common.  I would say that we would need a DiagnosticAssemblerKind.td for these.


https://reviews.llvm.org/D27360