[PATCH] Make local: optional.

Rafael EspĂ­ndola via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 17 05:29:10 PDT 2016


Updated version with Peter's suggestion.

Cheers,
Rafael


On 16 June 2016 at 19:22, Peter Collingbourne <peter at pcc.me.uk> wrote:
> Or we could just rename/reverse the sense of the existing VersionScript flag
> -- we aren't using it for anything else.
>
> Peter
>
> On Thu, Jun 16, 2016 at 3:43 PM, Rui Ueyama via llvm-commits
> <llvm-commits at lists.llvm.org> wrote:
>>
>> I think we should add a new member to Config (say, `bool
>> VersionScriptGlobalByDefault`) to manage "existence of version script" and
>> "whether symbols are hidden or global by default" separately.
>>
>> On Thu, Jun 16, 2016 at 2:53 PM, Rafael EspĂ­ndola
>> <rafael.espindola at gmail.com> wrote:
>>>
>>> Doing that in an anonymous version is a bit silly, but this opens the
>>> way for supporting it in general.
>>>
>>> Since we don't support actual versions, for now we just disable the
>>> version script if we detect that it is missing a local.
>>>
>>> Cheers,
>>> Rafael
>>
>>
>>
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>>
>
>
>
> --
> --
> Peter
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Make-local-optional.patch
Type: application/octet-stream
Size: 5411 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160617/8e5e193f/attachment.obj>


More information about the llvm-commits mailing list