<div dir="ltr"><div>Nice, LGTM!  =D</div><div><br></div><div>I assume the Target/X86/X86ISelLowering.cpp changes are unrelated and are for a separate commit.</div><div><br></div>It seems that once we do this, our assembly will no longer be consumable by GNU as, which I guess is why you turned on integrated-as for mingw recently.  Do you see that as a problem, or is gas just hopelessly broken for COFF because it can't represent what we're trying to assemble here?</div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Nov 26, 2013 at 3:26 PM, Rafael Espíndola <span dir="ltr"><<a href="mailto:rafael.espindola@gmail.com" target="_blank">rafael.espindola@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">ccing the correct list and attaching the correct patch this time :-(<br>
<div class="HOEnZb"><div class="h5"><br>
<br>
On 26 November 2013 17:57, Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br>
> With the attached patch we use simple names for COMDAT sections (like<br>
> .text or .bss). This matches the MSVC behavior.<br>
><br>
> When merging it is the COMDAT symbol that is used to decide if two<br>
> sections should be merged, so there is no point in building a fancy<br>
> name.<br>
><br>
> This survived a bootstrap on mingw32.<br>
><br>
> Cheers,<br>
> Rafael<br>
</div></div></blockquote></div><br></div>