<br><br><div class="gmail_quote">On Sat, May 8, 2010 at 11:36 AM, Chris Lattner <span dir="ltr"><<a href="mailto:clattner@apple.com">clattner@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
On May 7, 2010, at 4:58 PM, Nathan Jeffords wrote:<br>
<br>
> here is a patch that adds EmitCOFFSymbolDef to the MCStreamer interface this allows a symbols storage class & type to be set from the AsmPrinter without using EmitRawText. <EmitCOFFSymbolDef.patch><br>
<br>
Are you sure that this is better than emitting each directive individually?  Should def/scl/type/endef each be their own callbacks?<br>
<br>
The MC design assumes that we want to build real native assemblers (if for no other reason, this makes it really easy to verify that the -filetype=obj and -filetype=asm paths are generating the same code).  If you start aggregating callbacks like this, we can't do this reasonably.<br>

<br>
Another way of asking the question: if gas can handle these directives individually, why does the COFF emitter need them aggregated together?<br>
<font color="#888888"><br>
-Chris</font></blockquote></div><br><div>OK, I reworked the patch to break out the components of the symbol definition into separate callbacks.</div>