[PATCH] D92200: Ensure SplitEdge to return the new block between the two given blocks
    Bangtian Liu via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Dec 16 00:06:14 PST 2020
    
    
  
kobeliu85 added inline comments.
================
Comment at: llvm/unittests/Transforms/Utils/BasicBlockUtilsTest.cpp:281
+
+  ASSERT_DEATH(
+      {
----------------
Meinersbur wrote:
> kaz7 wrote:
> > FYI, having failed to die error here.  I appreciate if you fix it.
> I think the crash would be due to an `assert`, which are removed in release builds. We cannot directly test assertions.
Thanks for the comment, I will investigate this today. 
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D92200/new/
https://reviews.llvm.org/D92200
    
    
More information about the llvm-commits
mailing list