[PATCH] D65287: [analyzer][CFG] Don't track the condition of asserts
    Kristóf Umann via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Sat Aug 17 10:06:53 PDT 2019
    
    
  
Szelethus marked an inline comment as done.
Szelethus added inline comments.
================
Comment at: cfe/trunk/lib/StaticAnalyzer/Core/BugReporterVisitors.cpp:1759
+    if (isa<BinaryOperator>(ElseCond)) {
+      assert(cast<BinaryOperator>(ElseCond)->isLogicalOp());
+      return isAssertlikeBlock(Else, Context);
----------------
Okay, this is stupid. Commited rL369195 to turn this into a condition, accompanied with a test case that would cause an assertion failure.
Repository:
  rL LLVM
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65287/new/
https://reviews.llvm.org/D65287
    
    
More information about the cfe-commits
mailing list