[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