diff options
-rw-r--r-- | clang/lib/AST/ASTDiagnostic.cpp | 11 | ||||
-rw-r--r-- | clang/test/Misc/diag-template-diffing.cpp | 37 |
2 files changed, 45 insertions, 3 deletions
diff --git a/clang/lib/AST/ASTDiagnostic.cpp b/clang/lib/AST/ASTDiagnostic.cpp index 6e8dfcb937d..658d850b12b 100644 --- a/clang/lib/AST/ASTDiagnostic.cpp +++ b/clang/lib/AST/ASTDiagnostic.cpp @@ -990,19 +990,22 @@ class TemplateDiff { } }; + bool UseDesugaredIterator; InternalIterator SugaredIterator; InternalIterator DesugaredIterator; public: TSTiterator(ASTContext &Context, const TemplateSpecializationType *TST) - : SugaredIterator(TST), + : UseDesugaredIterator(TST->isSugared() && !TST->isTypeAlias()), + SugaredIterator(TST), DesugaredIterator( GetTemplateSpecializationType(Context, TST->desugar())) {} /// &operator++ - Increment the iterator to the next template argument. TSTiterator &operator++() { ++SugaredIterator; - ++DesugaredIterator; + if (UseDesugaredIterator) + ++DesugaredIterator; return *this; } @@ -1024,11 +1027,13 @@ class TemplateDiff { /// hasDesugaredTA - Returns true if there is another TemplateArgument /// available. bool hasDesugaredTA() const { - return !DesugaredIterator.isEnd(); + return UseDesugaredIterator && !DesugaredIterator.isEnd(); } /// getDesugaredTA - Returns the desugared TemplateArgument. reference getDesugaredTA() const { + assert(UseDesugaredIterator && + "Desugared TemplateArgument should not be used."); return *DesugaredIterator; } }; diff --git a/clang/test/Misc/diag-template-diffing.cpp b/clang/test/Misc/diag-template-diffing.cpp index 5e1caa5c31c..90bcf6b2d17 100644 --- a/clang/test/Misc/diag-template-diffing.cpp +++ b/clang/test/Misc/diag-template-diffing.cpp @@ -1421,6 +1421,43 @@ B<const A<>> b4 = B<>(); // CHECK-ELIDE-NOTREE: error: no viable conversion from 'B<A<...>>' to 'B<const A<...>>' } +namespace TypeAlias { + +template <typename T> class vector {}; + +template <int Dimension> class Point; +template <int dimension, typename T> using Polygon = vector<Point<dimension>>; + +void foo(Polygon<3, float>); +void bar() { foo(Polygon<2, float>()); } + +// CHECK-ELIDE-NOTREE: error: no matching function for call to 'foo' +// CHECK-ELIDE-NOTREE: note: candidate function not viable: no known conversion from 'Polygon<2, [...]>' to 'Polygon<3, [...]>' for 1st argument + +enum class X { + X1, + X2, +}; + +template<X x> struct EnumToType; + +template <> struct EnumToType<X::X1> { using type = int; }; + +template <> struct EnumToType<X::X2> { using type = double; }; + + +template <X x> using VectorType = vector<typename EnumToType<x>::type>; + +template <X x> void D(const VectorType<x>&); + +void run() { + D<X::X1>(VectorType<X::X2>()); +} +// CHECK-ELIDE-NOTREE: error: no matching function for call to 'D' +// CHECK-ELIDE-NOTREE: note: candidate function [with x = TypeAlias::X::X1] not viable: no known conversion from 'VectorType<X::X2>' to 'const VectorType<(TypeAlias::X)0>' for 1st argument + +} + // CHECK-ELIDE-NOTREE: {{[0-9]*}} errors generated. // CHECK-NOELIDE-NOTREE: {{[0-9]*}} errors generated. // CHECK-ELIDE-TREE: {{[0-9]*}} errors generated. |