Put more stuff in the comdat used for variables with static init

Reid Kleckner rnk at google.com
Thu Sep 18 16:35:02 PDT 2014


We should also omit the third entry in @llvm.global_ctors.

On Thu, Sep 18, 2014 at 4:31 PM, David Majnemer <david.majnemer at gmail.com>
wrote:

> LGTM.
>
> On Thu, Sep 18, 2014 at 3:53 PM, Rafael EspĂ­ndola <
> rafael.espindola at gmail.com> wrote:
>
>> On 16 September 2014 12:28, David Majnemer <david.majnemer at gmail.com>
>> wrote:
>> > I think we should have clang omit comdats, it would be confusing to
>> silently
>> > ignore comdats in llvm.
>> >
>>
>> Yes, that is probably best.
>>
>> What about the attached patch?
>>
>> Cheers,
>> Rafael
>>
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140918/9cf5f6d2/attachment.html>


More information about the cfe-commits mailing list