<div dir="ltr">Ok sounds good.<div><br></div><div>David</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, May 25, 2016 at 9:38 AM, Reid Kleckner <span dir="ltr"><<a href="mailto:rnk@google.com" target="_blank">rnk@google.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 Wed, May 25, 2016 at 9:32 AM, Xinliang David Li <<a href="mailto:davidxl@google.com">davidxl@google.com</a>> wrote:<br>
> On Wed, May 25, 2016 at 9:29 AM, Reid Kleckner <<a href="mailto:rnk@google.com">rnk@google.com</a>> wrote:<br>
>><br>
>> +  if (Triple(M->getTargetTriple()).isOSBinFormatCOFF())<br>
>> +    User->setComdat(M->getOrInsertComdat(User->getName()));<br>
>><br>
>> I think this should really be 'if (!Triple(...).isOSBinFormatMachO())<br>
>> ...->setComdat...', i.e. we should always set the comdat if the<br>
>> platform supports it.<br>
><br>
> Is there a more generic query for that ?<br>
<br>
</span>Not yet, but we should probably just hoist this method from clang up<br>
to the triple:<br>
  bool CodeGenModule::supportsCOMDAT() const {<br>
    return !getTriple().isOSBinFormatMachO();<br>
  }<br>
</blockquote></div><br></div>