[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