Re: [PATCH] D13664: [Fix] Don't emit multiple diagnostics for the same error

2015-10-12 Thread George Burgess IV via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL250078: [Sema] Don't emit multiple diags for one error (authored by gbiv). Changed prior to commit: http://reviews.llvm.org/D13664?vs=37137&id=37138#toc Repository: rL LLVM http://reviews.llvm.org/D

Re: [PATCH] D13664: [Fix] Don't emit multiple diagnostics for the same error

2015-10-12 Thread Richard Smith via cfe-commits
rsmith accepted this revision. rsmith added a comment. This revision is now accepted and ready to land. LGTM, thanks! http://reviews.llvm.org/D13664 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listin

Re: [PATCH] D13664: [Fix] Don't emit multiple diagnostics for the same error

2015-10-12 Thread George Burgess IV via cfe-commits
george.burgess.iv updated this revision to Diff 37137. george.burgess.iv added a comment. Un-nested an if statement. http://reviews.llvm.org/D13664 Files: lib/Sema/SemaOverload.cpp test/SemaCXX/addr-of-overloaded-function.cpp Index: test/SemaCXX/addr-of-overloaded-function.cpp

[PATCH] D13664: [Fix] Don't emit multiple diagnostics for the same error

2015-10-12 Thread George Burgess IV via cfe-commits
george.burgess.iv created this revision. george.burgess.iv added a reviewer: rsmith. george.burgess.iv added a subscriber: cfe-commits. Given code like the following (stolen from tests): ``` template int f(T1 *, const T2 *); template int f(const T1 *, T2 *); int (*p)(const int *, const int *) =