[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