[PATCH] D124450: [AMDGPU] Remove hasOneUse check from scalar select pattern

[Piotr Sobczak via Phabricator via llvm-commits]

piotr added a comment.

The biggest headache comes from the fact that during `moveToVALU` when `S_CMP` gets converted to `V_CMP` the users of SCC need to be handled properly otherwise you end up with a weird copy from SCC. I think this is handled right now by adding an extra copy from VCC to SCC to make the connection between `V_CMP` and `S_CSELECT` until it is time for the handling of `S_CSELECT`'. This gets more tricky when there are more uses of SCC I guess.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124450