Re: C++11: new builtin to allow constexpr to be applied to performance-critical functions

2012-10-20 Thread Andy Gibbs
On Saturday, October 20, 2012 7:50 AM, Chandler Carruth wrote: [...snip...] Let me hypothesize a different interface: This stays the same... constexpr int constexpr_strncmp(const char *p, const char *q, size_t n) { return !n ? 0 : *p != *q ? *p - *q : !*p ? 0 : constexpr_strncmp(p+1, q+1,

[C++ Patch] __builtin_choose_expr *bump*

2011-10-21 Thread Andy Gibbs
Hi, Please can I bump this patch and ask for it to be approved and committed: http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01711.html The patch is to implement the C built-in function __builtin_choose_expr(...) in C++. I'm afraid I am new to contributing to GCC, so I hope I am going about this

Re: [C++ Patch] __builtin_choose_expr *bump*

2011-10-21 Thread Andy Gibbs
On Friday, October 21, 2011 4:42 PM, Joseph S. Myers wrote: On Fri, 21 Oct 2011, Andy Gibbs wrote: Hi, Please can I bump this patch and ask for it to be approved and committed: http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01711.html Have you sent in your copyright assignment papers

Re: [C++ Patch] __builtin_choose_expr

2011-10-19 Thread Andy Gibbs
Andy Gibbs wrote on 18th October 2011 at 1:36 PM: Paolo Carlini wrote on 18th October 2011 at 12:00 PM: I'm under the impression that some tests are written in C, wouldn't better fit in c-c++-common? Ok, I can do this but I think only 2 out of the 8 tests can be moved across. I

[C++ Patch] __builtin_choose_expr

2011-10-18 Thread Andy Gibbs
# of unresolved testcases 1 # of unsupported tests 160 This is my first patch contributed to GCC. I hope that I have met the required quality standard! I will be happy to answer any questions / reformat the patch as necessary... Regards, Andy = Changelog: 2011-10-17 Andy

Re: [C++ Patch] __builtin_choose_expr

2011-10-18 Thread Andy Gibbs
- Original Message - From: Paolo Carlini paolo.carl...@oracle.com To: Andy Gibbs andyg1...@hotmail.co.uk Cc: gcc-patches@gcc.gnu.org; i...@google.com Sent: Tuesday, October 18, 2011 12:00 PM Subject: Re: [C++ Patch] __builtin_choose_expr I'm under the impression that some tests