[llvm] r241466 - Resubmit "Add new EliminateAvailableExternally module pass" (r239480)

Teresa Johnson tejohnson at google.com
Mon Jul 13 08:38:35 PDT 2015


Thanks, Rafael!

Question about your follow up commit r242032. Why is the NumAliases
unused? I don't see the update to that being removed in your other
commits.

Teresa

On Mon, Jul 13, 2015 at 7:45 AM, Rafael EspĂ­ndola
<rafael.espindola at gmail.com> wrote:
> On 11 July 2015 at 12:58, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:
>> Having to change the visibility in here is really odd.
>>
>> Reid, your theory was that clang was not setting the visibility
>> correctly, was that not the case?
>
>
> Fixed in r242012, r242027 and r242030.
>
> Cheers,
> Rafael



-- 
Teresa Johnson | Software Engineer | tejohnson at google.com | 408-460-2413




More information about the llvm-commits mailing list