<div dir="ltr">Please use llvm_unreachable at the bottom of shouldBeInCOMDAT, gcc warns otherwise. LGTM.</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Jan 8, 2015 at 6:00 AM, 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"><span class="">On 8 January 2015 at 03:42, David Majnemer <<a href="mailto:david.majnemer@gmail.com">david.majnemer@gmail.com</a>> wrote:<br>
> I don't think we'd want to keep marking C++ entities as weak/linkonce/etc.<br>
> in the MSVC ABI if we are splitting the COMDATness out.  I guess we still<br>
> sorta need it to talk about the discardability of a symbol...<br>
<br>
</span>And that should really be an independent patch, no? It is basically<br>
the "splitting linkage" discussion and it is larger than this.<br>
<br>
Cheers,<br>
Rafael<br>
</blockquote></div><br></div>