[PATCH] D54355: Use is.constant intrinsic for __builtin_constant_p

[Nick Desaulniers via Phabricator via cfe-commits]

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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits