[PATCH] D52221: require lower-switch in preISel passes for AMDGPU
Nicolai Hähnle via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 18 04:10:38 PDT 2018
nhaehnle added a comment.
Good to hear the new DA can handle this. We need lowered switches for the control flow lowering anyway, though, so we may as well do it this way.
The test case is extremely mysterious to me, though. What is it that's actually being tested here? From the description I understand it's the divergence of the phi in sw.epilog? I don't see how that relates to the CHECK lines.
Repository:
rL LLVM
https://reviews.llvm.org/D52221
More information about the llvm-commits
mailing list