[PATCH] D39993: Remove dead code

[Peter Collingbourne via llvm-commits]

The only guess that we had was that the bot was under such high load that
it exceeded the retry limits in the new rename implementation.

Peter

On Mon, Nov 13, 2017 at 7:08 PM, Rafael Avila de Espindola <
rafael.espindola at gmail.com> wrote:

> Any guesses as to what causes the failure with the new implementation?
>
> Cheers,
> Rafael
>
> Peter Collingbourne via Phabricator <reviews at reviews.llvm.org> writes:
>
> > pcc added a subscriber: inglorion.
> > pcc added a comment.
> >
> > Unfortunately, we were still seeing occasional permission denied errors
> on our bots even with that change.
> > e.g. https://build.chromium.org/p/chromium.clang/builders/
> ToTWinThinLTO64/builds/101
> > So this code was added as part of https://reviews.llvm.org/D39874.
> >
> > The confusion is understandable. Bob, can you please update the comment
> to make it clear why we need that code even with a better implementation of
> rename?
> >
> >
> > https://reviews.llvm.org/D39993
>



-- 
-- 
Peter
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171115/6b96f955/attachment.html>