diff options
| author | Richard Smith <richard-llvm@metafoo.co.uk> | 2017-10-28 01:15:00 +0000 |
|---|---|---|
| committer | Richard Smith <richard-llvm@metafoo.co.uk> | 2017-10-28 01:15:00 +0000 |
| commit | cb18957da05afab4c15411fca2dc13b85339ef2b (patch) | |
| tree | add3037004b0da501a0a525f62bffab3d1cdb949 | |
| parent | 49687104d6afed98e2eb5d90129d196a4c587add (diff) | |
| download | bcm5719-llvm-cb18957da05afab4c15411fca2dc13b85339ef2b.tar.gz bcm5719-llvm-cb18957da05afab4c15411fca2dc13b85339ef2b.zip | |
Never try to instantiate a deduction guide's "definition". Fixes bogus warning when there inevitably isn't one.
llvm-svn: 316820
| -rw-r--r-- | clang/lib/Sema/SemaTemplateInstantiateDecl.cpp | 3 | ||||
| -rw-r--r-- | clang/test/SemaCXX/cxx1z-class-template-argument-deduction.cpp | 4 |
2 files changed, 4 insertions, 3 deletions
diff --git a/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp b/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp index d93fbd7c830..5b528fa9c2e 100644 --- a/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp +++ b/clang/lib/Sema/SemaTemplateInstantiateDecl.cpp @@ -3768,7 +3768,8 @@ void Sema::InstantiateFunctionDefinition(SourceLocation PointOfInstantiation, bool Recursive, bool DefinitionRequired, bool AtEndOfTU) { - if (Function->isInvalidDecl() || Function->isDefined()) + if (Function->isInvalidDecl() || Function->isDefined() || + isa<CXXDeductionGuideDecl>(Function)) return; // Never instantiate an explicit specialization except if it is a class scope diff --git a/clang/test/SemaCXX/cxx1z-class-template-argument-deduction.cpp b/clang/test/SemaCXX/cxx1z-class-template-argument-deduction.cpp index 9232a8b6eba..9080f67fe0e 100644 --- a/clang/test/SemaCXX/cxx1z-class-template-argument-deduction.cpp +++ b/clang/test/SemaCXX/cxx1z-class-template-argument-deduction.cpp @@ -1,5 +1,5 @@ -// RUN: %clang_cc1 -std=c++1z -verify %s -DERRORS -// RUN: %clang_cc1 -std=c++1z -verify %s -UERRORS +// RUN: %clang_cc1 -std=c++1z -verify %s -DERRORS -Wundefined-func-template +// RUN: %clang_cc1 -std=c++1z -verify %s -UERRORS -Wundefined-func-template // This test is split into two because we only produce "undefined internal" // warnings if we didn't produce any errors. |
