[PATCH] D60834: [AMDGPU] Uniform values being used outside loop marked non-divergent
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Apr 17 13:49:05 PDT 2019
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
More information about the llvm-commits
mailing list