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

Ruiling, Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 22 23:43:29 PDT 2022


ruiling added a comment.

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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134439



More information about the llvm-commits mailing list