Re: C++ PATCH for c++/81676 - bogus -Wunused warnings in constexpr if

2019-08-27 Thread Jason Merrill
On Tue, Aug 27, 2019 at 4:10 PM Marek Polacek wrote: > On Fri, Aug 23, 2019 at 02:20:53PM -0700, Jason Merrill wrote: > > On 8/19/19 11:28 AM, Marek Polacek wrote: > > > On Fri, Aug 16, 2019 at 06:20:20PM -0700, Jason Merrill wrote: > > > > On 8/16/19 2:29 PM, Marek Polacek wrote: > > > > > This

Re: C++ PATCH for c++/81676 - bogus -Wunused warnings in constexpr if

2019-08-27 Thread Marek Polacek
On Fri, Aug 23, 2019 at 02:20:53PM -0700, Jason Merrill wrote: > On 8/19/19 11:28 AM, Marek Polacek wrote: > > On Fri, Aug 16, 2019 at 06:20:20PM -0700, Jason Merrill wrote: > > > On 8/16/19 2:29 PM, Marek Polacek wrote: > > > > This patch is an attempt to fix the annoying -Wunused-but-set-* > >

Re: C++ PATCH for c++/81676 - bogus -Wunused warnings in constexpr if

2019-08-23 Thread Jason Merrill
On 8/19/19 11:28 AM, Marek Polacek wrote: On Fri, Aug 16, 2019 at 06:20:20PM -0700, Jason Merrill wrote: On 8/16/19 2:29 PM, Marek Polacek wrote: This patch is an attempt to fix the annoying -Wunused-but-set-* warnings that tend to occur with constexpr if. When we have something like

Re: C++ PATCH for c++/81676 - bogus -Wunused warnings in constexpr if

2019-08-19 Thread Marek Polacek
On Fri, Aug 16, 2019 at 06:20:20PM -0700, Jason Merrill wrote: > On 8/16/19 2:29 PM, Marek Polacek wrote: > > This patch is an attempt to fix the annoying -Wunused-but-set-* warnings > > that > > tend to occur with constexpr if. When we have something like > > > >template < typename T > > >

Re: C++ PATCH for c++/81676 - bogus -Wunused warnings in constexpr if

2019-08-16 Thread Jason Merrill
On 8/16/19 2:29 PM, Marek Polacek wrote: This patch is an attempt to fix the annoying -Wunused-but-set-* warnings that tend to occur with constexpr if. When we have something like template < typename T > int f(T v){ if constexpr(sizeof(T) == sizeof(int)){ return v;

C++ PATCH for c++/81676 - bogus -Wunused warnings in constexpr if

2019-08-16 Thread Marek Polacek
This patch is an attempt to fix the annoying -Wunused-but-set-* warnings that tend to occur with constexpr if. When we have something like template < typename T > int f(T v){ if constexpr(sizeof(T) == sizeof(int)){ return v; }else{ return 0; } } and