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

Nick Desaulniers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 4 13:42:07 PDT 2018


nickdesaulniers added inline comments.


================
Comment at: test/Sema/builtins.c:125
 
+// __builtin_constant_p cannot resolve non-constants as a file scoped array.
+int expr;
----------------
such as

Otherwise it's possible to read this as the non-constants being resolved to a file scoped array.


https://reviews.llvm.org/D52854





More information about the cfe-commits mailing list