[PATCH] D87585: [AMDGPU] Dynamically clear renamable to avoid constant bus errors

[Jay Foad via Phabricator via llvm-commits]

foad added a comment.

In D87585#2276099 <https://reviews.llvm.org/D87585#2276099>, @critson wrote:

> In D87585#2276097 <https://reviews.llvm.org/D87585#2276097>, @foad wrote:
>
>> Are there any other significant changes? I'm thinking of things like this that would help with dependency stalls on gfx10:
>>
>>   v_mov v0, 0
>>   v_mov v1, v0
>>
>> ->
>>
>>   v_mov v0, 0
>>   v_mov v1, 0



> I will review shader changes; however, I do not think Machine Copy Propagation propagates constants.

True. But this can also help with stalls:

  v_mov v0, s0
  v_mov v1, v0

->

  v_mov v0, s0
  v_mov v1, s0


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87585/new/

https://reviews.llvm.org/D87585