[PATCH] D52221: require lower-switch in preISel passes for AMDGPU

Sameer Sahasrabuddhe via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 18 10:46:58 PDT 2018


sameerds added a comment.

In https://reviews.llvm.org/D52221#1238073, @simoll wrote:

> This workaround will not longer be required with the new Divergence Analysis (https://reviews.llvm.org/D50433). I will add this test case as a unit test for the new implementation.


Thanks @simoll. The new unit test in https://reviews.llvm.org/D50433 correctly captures the issue.


Repository:
  rL LLVM

https://reviews.llvm.org/D52221





More information about the llvm-commits mailing list