Re: r257710 - [Sema] Suppress diags in overload resolution.

2016-02-01 Thread Hans Wennborg via cfe-commits
Thanks! r259412. On Mon, Feb 1, 2016 at 1:32 PM, Richard Smith wrote: > LGTM for 3.8. > > On Wed, Jan 13, 2016 at 4:03 PM, Hans Wennborg wrote: >> Thanks! Richard, what say the code owner? >> >> On Wed, Jan 13, 2016 at 3:59 PM, George Burgess IV >> wrote: >>> FYI: This patch should to be merged

Re: r257710 - [Sema] Suppress diags in overload resolution.

2016-02-01 Thread Richard Smith via cfe-commits
LGTM for 3.8. On Wed, Jan 13, 2016 at 4:03 PM, Hans Wennborg wrote: > Thanks! Richard, what say the code owner? > > On Wed, Jan 13, 2016 at 3:59 PM, George Burgess IV > wrote: >> FYI: This patch should to be merged into the clang 3.8.0 branch. >> >> Thank you! >> George >> >> On Wed, Jan 13, 201

Re: r257710 - [Sema] Suppress diags in overload resolution.

2016-01-13 Thread Hans Wennborg via cfe-commits
Thanks! Richard, what say the code owner? On Wed, Jan 13, 2016 at 3:59 PM, George Burgess IV wrote: > FYI: This patch should to be merged into the clang 3.8.0 branch. > > Thank you! > George > > On Wed, Jan 13, 2016 at 3:36 PM, George Burgess IV via cfe-commits > wrote: >> >> Author: gbiv >> Dat

Re: r257710 - [Sema] Suppress diags in overload resolution.

2016-01-13 Thread George Burgess IV via cfe-commits
FYI: This patch should to be merged into the clang 3.8.0 branch. Thank you! George On Wed, Jan 13, 2016 at 3:36 PM, George Burgess IV via cfe-commits < cfe-commits@lists.llvm.org> wrote: > Author: gbiv > Date: Wed Jan 13 17:36:34 2016 > New Revision: 257710 > > URL: http://llvm.org/viewvc/llvm-p

r257710 - [Sema] Suppress diags in overload resolution.

2016-01-13 Thread George Burgess IV via cfe-commits
Author: gbiv Date: Wed Jan 13 17:36:34 2016 New Revision: 257710 URL: http://llvm.org/viewvc/llvm-project?rev=257710&view=rev Log: [Sema] Suppress diags in overload resolution. We were emitting diagnostics from our shiny new C-only overload resolution mode. This patch attempts to silence all such