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

Nick Desaulniers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Nov 20 09:47:00 PST 2018


nickdesaulniers added inline comments.


================
Comment at: lib/AST/ExprConstant.cpp:11391
 
   llvm_unreachable("Invalid StmtClass!");
 }
----------------
@void , I assume this unreachable is the one reported by @uweigand ? Does the above switch need a case statement added?


Repository:
  rC Clang

https://reviews.llvm.org/D54355





More information about the cfe-commits mailing list