[PATCH] D54355: Use is.constant intrinsic for __builtin_constant_p
    Nick Desaulniers via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Mon Nov 19 15:54:37 PST 2018
    
    
  
nickdesaulniers added inline comments.
================
Comment at: lib/StaticAnalyzer/Core/ExprEngine.cpp:1285
 
     case Expr::ConstantExprClass:
     case Stmt::ExprWithCleanupsClass:
----------------
nickdesaulniers wrote:
> `LLVM_FALLTHROUGH;` ?
nvm, only needed when there's code THEN fallthrough.
Repository:
  rC Clang
https://reviews.llvm.org/D54355
    
    
More information about the cfe-commits
mailing list