[cfe-commits] r146236 - in /cfe/trunk: lib/AST/ExprConstant.cpp test/Sema/builtins.c

[Richard Smith]

On Fri, December 9, 2011 02:35, Chandler Carruth wrote:
> On Thu, Dec 8, 2011 at 6:04 PM, Richard Smith
> <richard-llvm at metafoo.co.uk>wrote:
>> Replace the implementation of __builtin_constant_p (which was based on the GCC
>> documentation) with one based on what GCC's __builtin_constant_p is actually
>> intended to do (discovered by asking a friendly GCC developer).
>
> Please document what this actually does in the extensions document! =D

I've updated the text explaining __builtin_constant_p's semantics in the
internals manual in change r146241. Do you also want a section in the language
extensions document? I worry that documenting this too publicly will encourage
people to rely on the precise behaviour.

- Richard