[PATCH] D134439: [StructurizeCFG] Remove imposible case, associated test, and replace by assert

[Juan Manuel Martinez Caamaño via Phabricator via llvm-commits]

jmmartinez added a comment.

In D134439#3810796 <https://reviews.llvm.org/D134439#3810796>, @ruiling wrote:

> I think it should be ok to remove the code, but I would rather we keep the tests, the test is hitting the limitation of StructurizeCFG. The test contains infinite loop in its CFG. Maybe we can update the comment in the test to say this, and see whether we could solve this problem later.

Whay do you think about removing this test and introducing a separate `infinite-loop.ll` test ? I'd prefer a test that matches what we expect `StructurizeCFG` to do on an infinite loop (in this particular case, nothing) than the current test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134439