[PATCH] D133593: [AMDGPU] Always select s_cselect_b32 for uniform 'select' SDNode

[Jay Foad via Phabricator via llvm-commits]

foad added a comment.

> This patch contains changes necessary to carry physical condition register (SCC) dependencies through the SDNode scheduler. This allows avoiding inserting an SCC copy between each definition and use.

Please explain more. Why doesn't the generic support for physical registers work already? Why do we need to "avoid inserting an SCC copy"?

I was also working on this in D124450 <https://reviews.llvm.org/D124450> but I have not looked at that patch for a few months.


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

https://reviews.llvm.org/D133593