[PATCH] D39993: Remove dead code

Rafael Avila de Espindola via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 13 19:08:54 PST 2017


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


More information about the llvm-commits mailing list