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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to