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 can

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 b