[PATCH] D62937: [AMDGPU] Mark s_endpgm convergent

[Nicolai Hähnle via Phabricator via llvm-commits]

nhaehnle added a comment.

I also don't understand the purpose of this and what this solves.

Convergent is defined as "may not be modified to have additional control dependencies". However, at the MIR level our branches are all effectively scalar, so adding control dependencies should actually always be okay.


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

https://reviews.llvm.org/D62937