[PATCH] D60834: [AMDGPU] Uniform values being used outside loop marked non-divergent

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added a comment.

In D60834#1470763 <https://reviews.llvm.org/D60834#1470763>, @arsenm wrote:

> It sort of intuitively makes sense to me that the control flow lowering would like LCSSA. However, this should not be handled by adding it directly to the pass pipeline. You can add this as a dependency, e.g.     AU.addRequiredID(LCSSAID);
>
> I would also like to see the an IR->IR testcase showing LCSSA was implicitly run


Actually, what really requires LCSSA? Is it DivergenceAnalysis or StructurizeCFG directly?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60834