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

James Y Knight via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 12 19:54:09 PST 2018


jyknight added a comment.

That example cannot be expected to ever evaluate the expression as "1" -- it doesn't in GCC, nor should it in Clang. An asm constraint of "n" or "i" (but not, e.g., "nr") must require a constant expression, and evaluating the argument as a constant expression necessarily means always evaluating it to -1.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54355/new/

https://reviews.llvm.org/D54355





More information about the cfe-commits mailing list