Re: C++ PATCH to add test to cover case RANGE_FOR_STMT

2018-11-01 Thread Jason Merrill
On 11/1/18 2:06 PM, Marek Polacek wrote: Ping. On Thu, Oct 11, 2018 at 09:28:38PM -0400, Marek Polacek wrote: Recently it came up that no test in the testsuite triggers the RANGE_FOR_STMT case in potential_constant_expression_1. I came up with this valid test that tickles that codepath. I

Re: C++ PATCH to add test to cover case RANGE_FOR_STMT

2018-11-01 Thread Marek Polacek
Ping. On Thu, Oct 11, 2018 at 09:28:38PM -0400, Marek Polacek wrote: > Recently it came up that no test in the testsuite triggers the RANGE_FOR_STMT > case in potential_constant_expression_1. I came up with this valid test that > tickles that codepath. > > I can't use ({ }) instead of a lambda

C++ PATCH to add test to cover case RANGE_FOR_STMT

2018-10-11 Thread Marek Polacek
Recently it came up that no test in the testsuite triggers the RANGE_FOR_STMT case in potential_constant_expression_1. I came up with this valid test that tickles that codepath. I can't use ({ }) instead of a lambda because the constexpr machinery doesn't handle statement expressions; see