[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