<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On May 11, 2010, at 6:18 PM, Nathan Jeffords wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">I would like to commit this patch as part of my ongoing work on improving COFF support in the MC library.<div><br></div><div>This patch changes where the .linkonce attribute is emitted from the symbol to the section the symbol was assigned to.</div>
</blockquote></div><br><div>Looks great, please fix the brace here:  :-)</div><div><br></div><div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; font: normal normal normal 12px/normal Inconsolata; ">+  if (getCharacteristics() & MCSectionCOFF::IMAGE_SCN_LNK_COMDAT)</div><div style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0px; font: normal normal normal 12px/normal Inconsolata; ">+  {</div><div><font class="Apple-style-span" face="Inconsolata"><br></font></div><div><font class="Apple-style-span" face="Inconsolata">and commit!</font></div><div><font class="Apple-style-span" face="Inconsolata"><br></font></div><div><font class="Apple-style-span" face="Inconsolata">-Chris</font></div></div></body></html>