[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