diff options
| author | Douglas Gregor <dgregor@apple.com> | 2009-11-13 23:59:09 +0000 |
|---|---|---|
| committer | Douglas Gregor <dgregor@apple.com> | 2009-11-13 23:59:09 +0000 |
| commit | ff7028a55e8c8524dee4a86e3deda3d4a8ff2d88 (patch) | |
| tree | 60a44285fae6f94df5450d0cf6b1a7e120bdcf92 /clang/lib/Sema/SemaOverload.cpp | |
| parent | c5a332cefb53d9528e33c95694cc6bff9600ff3f (diff) | |
| download | bcm5719-llvm-ff7028a55e8c8524dee4a86e3deda3d4a8ff2d88.tar.gz bcm5719-llvm-ff7028a55e8c8524dee4a86e3deda3d4a8ff2d88.zip | |
Revert r88718, which does NOT solve the constructor-template-as-copy-constructor issue. Big thanks to John for finding this
llvm-svn: 88724
Diffstat (limited to 'clang/lib/Sema/SemaOverload.cpp')
| -rw-r--r-- | clang/lib/Sema/SemaOverload.cpp | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/clang/lib/Sema/SemaOverload.cpp b/clang/lib/Sema/SemaOverload.cpp index 70b080e678d..3948b22f7b6 100644 --- a/clang/lib/Sema/SemaOverload.cpp +++ b/clang/lib/Sema/SemaOverload.cpp @@ -2237,9 +2237,7 @@ Sema::AddOverloadCandidate(FunctionDecl *Function, // argument doesn't participate in overload resolution. } - // FIXME: It would be nice if it were safe to keep invalid methods in the - // overload set (but it isn't due to broken copy constructors). - if (!CandidateSet.isNewCandidate(Function) || Function->isInvalidDecl()) + if (!CandidateSet.isNewCandidate(Function)) return; // Add this candidate |
