Reverted as r<span style="line-height:1.5">226173.</span><br><div class="gmail_quote">On Thu Jan 15 2015 at 6:54:11 PM Rafael Espíndola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 15 January 2015 at 10:45, Timur Iskhodzhanov <<a href="mailto:timurrrr@google.com" target="_blank">timurrrr@google.com</a>> wrote:<br>
> Hi Rafael,<br>
><br>
> This change has completely broken down AddressSanitizer on Windows (ninja<br>
> check-asan), it fails to link anything due to multiple defined vftables,<br>
> e.g.<br>
><br>
> libcpmt.lib(xdateord.obj) : error LNK2005: "const<br>
> std::_System_error_category::`<u></u>vftable'"<br>
> (??_7_System_error_category@<u></u>std@@6B@) already defined in<br>
> sanitizer_allocator_test.cc.<u></u>i386.o<br>
><br>
> I'm going to revert your change.<br>
><br>
> Please coordinate with David and Kostya if you have any questions on how<br>
> COMDATs interplay with AddressSanitizer.<br>
<br>
Looks like it caused other comdat problems on windows. I am currently<br>
trying a bootstrap on mingw to debug it.<br>
<br>
Do revert it please, I didn't realized it was that bad.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div>