[PATCH] MCJIT honor symbol visibility settings when populating the global symbol table

Rafael EspĂ­ndola rafael.espindola at gmail.com
Mon Oct 20 11:17:37 PDT 2014


This patch looks good to me.

Lang, can you commit if the MCJIT bits look good to you?

On 20 October 2014 12:28, Anthony Pesch <inolen at gmail.com> wrote:
> I don't have commit access. Should I get that setup or can someone
> commit on my behalf?
>
> I've attached a patch without any spacing changes (sorry for the noise).
>
>  - Anthony
>
> On Mon, Oct 20, 2014 at 7:29 AM, Rafael EspĂ­ndola
> <rafael.espindola at gmail.com> wrote:
>> On 19 October 2014 18:16, Anthony Pesch <inolen at gmail.com> wrote:
>>> Hey guys,
>>>
>>> Alright, updated the patch, added isExportedToOtherDSO, updated
>>> comments, and converted the tests to spaces instead of tabs.
>>
>> The ELF bits looks good.
>>
>> Please commit the tabs to spaces change by itself before the rest of
>> the file. Do you have commit access?
>>
>> Cheers,
>> Rafael




More information about the llvm-commits mailing list