diff options
Diffstat (limited to 'clang')
-rw-r--r-- | clang/lib/AST/ODRHash.cpp | 20 | ||||
-rw-r--r-- | clang/test/Modules/odr_hash.cpp | 18 |
2 files changed, 37 insertions, 1 deletions
diff --git a/clang/lib/AST/ODRHash.cpp b/clang/lib/AST/ODRHash.cpp index 08593da89bb..b260bad9d37 100644 --- a/clang/lib/AST/ODRHash.cpp +++ b/clang/lib/AST/ODRHash.cpp @@ -140,7 +140,25 @@ void ODRHash::AddTemplateName(TemplateName Name) { } } -void ODRHash::AddTemplateArgument(TemplateArgument TA) {} +void ODRHash::AddTemplateArgument(TemplateArgument TA) { + auto Kind = TA.getKind(); + ID.AddInteger(Kind); + + switch (Kind) { + case TemplateArgument::Null: + case TemplateArgument::Declaration: + case TemplateArgument::NullPtr: + case TemplateArgument::Integral: + case TemplateArgument::Template: + case TemplateArgument::TemplateExpansion: + case TemplateArgument::Expression: + case TemplateArgument::Pack: + break; + case TemplateArgument::Type: + AddQualType(TA.getAsType()); + break; + } +} void ODRHash::AddTemplateParameterList(const TemplateParameterList *TPL) {} void ODRHash::clear() { diff --git a/clang/test/Modules/odr_hash.cpp b/clang/test/Modules/odr_hash.cpp index a6a0b74743a..fb7005be259 100644 --- a/clang/test/Modules/odr_hash.cpp +++ b/clang/test/Modules/odr_hash.cpp @@ -900,6 +900,24 @@ S2 s2; #endif } +namespace TemplateArgument { +#if defined(FIRST) +template<typename> struct U1 {}; +struct S1 { + U1<int> u; +}; +#elif defined(SECOND) +template<typename> struct U1 {}; +struct S1 { + U1<double> u; +}; +#else +S1 s1; +// expected-error@first.h:* {{'TemplateArgument::S1::u' from module 'FirstModule' is not present in definition of 'TemplateArgument::S1' in module 'SecondModule'}} +// expected-note@second.h:* {{declaration of 'u' does not match}} +#endif +} + // Interesting cases that should not cause errors. struct S should not error // while struct T should error at the access specifier mismatch at the end. namespace AllDecls { |