[PATCH] D87585: [AMDGPU] Dynamically clear renamable to avoid constant bus errors
Jay Foad via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Sep 14 07:41:54 PDT 2020
foad added a comment.
In D87585#2271185 <https://reviews.llvm.org/D87585#2271185>, @arsenm wrote:
> Where/why is renameable set in the first place? Can we just avoid setting it to begin with?
Looks like it was originally done to avoid passes like MachineCopyPropagation from introducing constant bus violations: rG1d531013876c02b18df678a5f67d6a7d94e392b9 <https://reviews.llvm.org/rG1d531013876c02b18df678a5f67d6a7d94e392b9>
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D87585/new/
https://reviews.llvm.org/D87585
More information about the llvm-commits
mailing list