[PATCH] D39993: Remove dead code

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 13 18:02:28 PST 2017


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