Re: C++ PATCH for c++/49813 (__builtin_isinf in constant expression)

2011-08-03 Thread Gabriel Dos Reis
On Mon, Aug 1, 2011 at 1:12 PM, Jason Merrill wrote: > Another issue raised in 49813 is that morally_constexpr_builtin_function_p > was rejecting __builtin_isinf because it has a variadic signature.  I'm > changing potential_constant_expression_1 to just allow any built-in function > in a constexp

Re: C++ PATCH for c++/49813 (__builtin_isinf in constant expression)

2011-08-01 Thread Paolo Carlini
Thanks a lot, Jason. I applied the below, tested x86_64-linux multilib. Paolo. // 2011-08-01 Paolo Carlini PR c++/49813 * include/c_global/cmath (isinf): Remove workaround. Index: include/c_global/cmath ===

Re: C++ PATCH for c++/49813 (__builtin_isinf in constant expression)

2011-08-01 Thread Jason Merrill
Another issue raised in 49813 is that morally_constexpr_builtin_function_p was rejecting __builtin_isinf because it has a variadic signature. I'm changing potential_constant_expression_1 to just allow any built-in function in a constexpr function; if it doesn't expand to a constant, we'll deal