Re: r329804 - [Sema] Fix built-in decrement operator overload resolution

2018-04-25 Thread Richard Smith via cfe-commits
On 12 April 2018 at 07:20, Jan Korous via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Hi Richard, > > Here you are: > https://reviews.llvm.org/D45569 > > I am now thinking if it makes sense to output this warning for pre-17 > standards: > > warning: incrementing expression of type bool is d

Re: r329804 - [Sema] Fix built-in decrement operator overload resolution

2018-04-12 Thread Jan Korous via cfe-commits
Hi Richard, Here you are: https://reviews.llvm.org/D45569 I am now thinking if it makes sense to output this warning for pre-17 standards: warning: incrementing expression of type bool is deprecated and incompatible with C++17 Produced in: SemaExpr.cpp stati

Re: r329804 - [Sema] Fix built-in decrement operator overload resolution

2018-04-11 Thread Richard Smith via cfe-commits
While you're here... ++ should not be available for bool in C++17 onwards. :) On Wed, 11 Apr 2018, 14:39 Jan Korous via cfe-commits, < cfe-commits@lists.llvm.org> wrote: > Author: jkorous > Date: Wed Apr 11 06:36:29 2018 > New Revision: 329804 > > URL: http://llvm.org/viewvc/llvm-project?rev=3298

r329804 - [Sema] Fix built-in decrement operator overload resolution

2018-04-11 Thread Jan Korous via cfe-commits
Author: jkorous Date: Wed Apr 11 06:36:29 2018 New Revision: 329804 URL: http://llvm.org/viewvc/llvm-project?rev=329804&view=rev Log: [Sema] Fix built-in decrement operator overload resolution C++ [over.built] p4: "For every pair (T, VQ), where T is an arithmetic type other than bool, and VQ is