Magic-Mirror/llvm-capstone
1
0
Fork 0
mirror of https://github.com/capstone-engine/llvm-capstone.git synced 2024-11-24 14:20:17 +00:00
Code Issues Actions 2 Packages Projects Releases Wiki Activity

PR39231: fix null dereference when diagnosing deduction failure due to

Browse Source 
conflicting values for a non-type pack.

llvm-svn: 344070
This commit is contained in:
Richard Smith 2018-10-09 18:49:22 +00:00
parent 81abca32fb
commit e54d9525ad
2 changed files with 17 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
clang/lib/Sema/SemaOverload.cpp
View File

@ -9984,7 +9984,7 @@ static void DiagnoseBadDeduction(Sema &S, NamedDecl *Found, Decl *Templated,
DeductionFailure.getFirstArg()->getNonTypeTemplateArgumentType();
QualType T2 =
DeductionFailure.getSecondArg()->getNonTypeTemplateArgumentType();
if (!S.Context.hasSameType(T1, T2)) {
if (!T1.isNull() && !T2.isNull() && !S.Context.hasSameType(T1, T2)) {
S.Diag(Templated->getLocation(),
diag::note_ovl_candidate_inconsistent_deduction_types)
<< ParamD->getDeclName() << *DeductionFailure.getFirstArg() << T1

16
clang/test/SemaTemplate/temp_arg_pack.cpp
View File

@ -6,3 +6,19 @@ namespace deduce_pack_non_pack {
template <typename T> void g(C<A<T>>); // expected-note {{candidate template ignored: deduced type 'C<A<[...], (no argument)>>' of 1st parameter does not match adjusted type 'C<A<[...], int>>' of argument [with T = bool]}}
void h(C<A<bool, int>> &x) { g(x); } // expected-error {{no matching function}}
}
namespace pr39231 {
template<typename T, T ...V> struct integer_sequence {};
template <typename T, T... A, T... B>
int operator^(integer_sequence<T, A...> a, // expected-note {{deduced conflicting values for parameter 'A' (<1, 2, 3> vs. <4, 5, 6>)}}
integer_sequence<T, A...> b);
int v = integer_sequence<int, 1, 2, 3>{} ^ integer_sequence<int, 4, 5, 6>{}; // expected-error {{invalid operands}}
template <typename T, T... A, T... B>
integer_sequence<T, A + B...> operator+(integer_sequence<T, A...> a,
integer_sequence<T, B...> b);
integer_sequence<int, 5, 7, 9> w =
integer_sequence<int, 1, 2, 3>{} + integer_sequence<int, 4, 5, 6>{};
}