Re: C++ PATCH for c++/87145 - bogus error converting class type in template argument list

2019-04-05 Thread Jason Merrill
On 4/5/19 4:50 PM, Marek Polacek wrote: On Fri, Apr 05, 2019 at 12:12:11AM -0400, Jason Merrill wrote: On 4/4/19 5:18 PM, Marek Polacek wrote: On Thu, Mar 28, 2019 at 03:54:30PM -0400, Jason Merrill wrote: On 3/20/19 4:12 PM, Marek Polacek wrote: The fix for 77656 caused us to call

Re: C++ PATCH for c++/87145 - bogus error converting class type in template argument list

2019-04-05 Thread Marek Polacek
On Fri, Apr 05, 2019 at 12:12:11AM -0400, Jason Merrill wrote: > On 4/4/19 5:18 PM, Marek Polacek wrote: > > On Thu, Mar 28, 2019 at 03:54:30PM -0400, Jason Merrill wrote: > > > On 3/20/19 4:12 PM, Marek Polacek wrote: > > > > The fix for 77656 caused us to call convert_nontype_argument even for >

Re: C++ PATCH for c++/87145 - bogus error converting class type in template argument list

2019-04-04 Thread Jason Merrill
On 4/4/19 5:18 PM, Marek Polacek wrote: On Thu, Mar 28, 2019 at 03:54:30PM -0400, Jason Merrill wrote: On 3/20/19 4:12 PM, Marek Polacek wrote: The fix for 77656 caused us to call convert_nontype_argument even for value-dependent arguments, to perform the conversion in order to avoid a bogus

Re: C++ PATCH for c++/87145 - bogus error converting class type in template argument list

2019-04-04 Thread Marek Polacek
On Thu, Mar 28, 2019 at 03:54:30PM -0400, Jason Merrill wrote: > On 3/20/19 4:12 PM, Marek Polacek wrote: > > The fix for 77656 caused us to call convert_nontype_argument even for > > value-dependent arguments, to perform the conversion in order to avoid > > a bogus warning. > > > > In this case,

Re: C++ PATCH for c++/87145 - bogus error converting class type in template argument list

2019-03-28 Thread Jason Merrill
On 3/20/19 4:12 PM, Marek Polacek wrote: The fix for 77656 caused us to call convert_nontype_argument even for value-dependent arguments, to perform the conversion in order to avoid a bogus warning. In this case, the argument is Pod{N}. The call to build_converted_constant_expr in

Re: C++ PATCH for c++/87145 - bogus error converting class type in template argument list

2019-03-27 Thread Marek Polacek
Ping. On Wed, Mar 20, 2019 at 04:12:13PM -0400, Marek Polacek wrote: > The fix for 77656 caused us to call convert_nontype_argument even for > value-dependent arguments, to perform the conversion in order to avoid > a bogus warning. > > In this case, the argument is Pod{N}. The call to >

C++ PATCH for c++/87145 - bogus error converting class type in template argument list

2019-03-20 Thread Marek Polacek
The fix for 77656 caused us to call convert_nontype_argument even for value-dependent arguments, to perform the conversion in order to avoid a bogus warning. In this case, the argument is Pod{N}. The call to build_converted_constant_expr in convert_nontype_argument produces Pod::operator