Re: r292497 - [Sema] Fix PR28181 by avoiding calling BuildOverloadedBinOp in C mode

2017-01-23 Thread Hans Wennborg via cfe-commits
On Thu, Jan 19, 2017 at 6:25 PM, Richard Smith wrote: > On 19 January 2017 at 15:52, Hans Wennborg wrote: >> >> Richard, what do you think? > > > Yes, let's merge this. r292808. Cheers, Hans >> On Thu, Jan 19, 2017 at 9:34 AM, Alex L wrote: >> > Hi Hans, >> > >> > Would it be possible to merg

Re: r292497 - [Sema] Fix PR28181 by avoiding calling BuildOverloadedBinOp in C mode

2017-01-19 Thread Richard Smith via cfe-commits
On 19 January 2017 at 15:52, Hans Wennborg wrote: > Richard, what do you think? Yes, let's merge this. > On Thu, Jan 19, 2017 at 9:34 AM, Alex L wrote: > > Hi Hans, > > > > Would it be possible to merge this for 4.0? > > > > Cheers, > > Alex > > > > On 19 January 2017 at 17:17, Alex Lorenz v

Re: r292497 - [Sema] Fix PR28181 by avoiding calling BuildOverloadedBinOp in C mode

2017-01-19 Thread Hans Wennborg via cfe-commits
Richard, what do you think? On Thu, Jan 19, 2017 at 9:34 AM, Alex L wrote: > Hi Hans, > > Would it be possible to merge this for 4.0? > > Cheers, > Alex > > On 19 January 2017 at 17:17, Alex Lorenz via cfe-commits > wrote: >> >> Author: arphaman >> Date: Thu Jan 19 11:17:57 2017 >> New Revision:

Re: r292497 - [Sema] Fix PR28181 by avoiding calling BuildOverloadedBinOp in C mode

2017-01-19 Thread Alex L via cfe-commits
Hi Hans, Would it be possible to merge this for 4.0? Cheers, Alex On 19 January 2017 at 17:17, Alex Lorenz via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: arphaman > Date: Thu Jan 19 11:17:57 2017 > New Revision: 292497 > > URL: http://llvm.org/viewvc/llvm-project?rev=292497&view

r292497 - [Sema] Fix PR28181 by avoiding calling BuildOverloadedBinOp in C mode

2017-01-19 Thread Alex Lorenz via cfe-commits
Author: arphaman Date: Thu Jan 19 11:17:57 2017 New Revision: 292497 URL: http://llvm.org/viewvc/llvm-project?rev=292497&view=rev Log: [Sema] Fix PR28181 by avoiding calling BuildOverloadedBinOp in C mode rdar://28532840 Differential Revision: https://reviews.llvm.org/D25213 Added: cfe/trun