[PATCH] D40556: SIFixSGPRCopies should not change non-divergent PHI

Alexander via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 10 06:35:02 PDT 2018


alex-t added a comment.

In fact yes. This is the most correct solution. The stuff in
SIFixSGPRCopies.cpp has got to gone.
So it does not make sense to spend efforts on it. Moreover there is no
correct solution at all
unless we implement one more DivergenceAnalysis upon the machine IR.
So, if it's okay I'd prefer to revert.


Repository:
  rL LLVM

https://reviews.llvm.org/D40556





More information about the llvm-commits mailing list