diff options
author | Benjamin Kramer <benny.kra@googlemail.com> | 2015-03-27 13:58:31 +0000 |
---|---|---|
committer | Benjamin Kramer <benny.kra@googlemail.com> | 2015-03-27 13:58:31 +0000 |
commit | 3b8044c02d330c8015e8f12079681b8d6b18fd2f (patch) | |
tree | 2da8d3d5acb2f9c31130d80a3c4e879949758f7d /clang/lib/Sema/SemaDeclCXX.cpp | |
parent | 39972f7723d23f6ca6e8d161941319e4da442439 (diff) | |
download | bcm5719-llvm-3b8044c02d330c8015e8f12079681b8d6b18fd2f.tar.gz bcm5719-llvm-3b8044c02d330c8015e8f12079681b8d6b18fd2f.zip |
[Sema] Diagnose default argument on a parameter pack.
This is ill-formed (and cannot be used anyways).
PR23028.
llvm-svn: 233376
Diffstat (limited to 'clang/lib/Sema/SemaDeclCXX.cpp')
-rw-r--r-- | clang/lib/Sema/SemaDeclCXX.cpp | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/clang/lib/Sema/SemaDeclCXX.cpp b/clang/lib/Sema/SemaDeclCXX.cpp index f7183bc2c4a..bb7ba943c60 100644 --- a/clang/lib/Sema/SemaDeclCXX.cpp +++ b/clang/lib/Sema/SemaDeclCXX.cpp @@ -316,8 +316,17 @@ Sema::ActOnParamDefaultArgument(Decl *param, SourceLocation EqualLoc, if (DiagnoseUnexpandedParameterPack(DefaultArg, UPPC_DefaultArgument)) { Param->setInvalidDecl(); return; - } - + } + + // C++11 [dcl.fct.default]p3 + // A default argument expression [...] shall not be specified for a + // parameter pack. + if (Param->isParameterPack()) { + Diag(EqualLoc, diag::err_param_default_argument_on_parameter_pack) + << DefaultArg->getSourceRange(); + return; + } + // Check that the default argument is well-formed CheckDefaultArgumentVisitor DefaultArgChecker(DefaultArg, this); if (DefaultArgChecker.Visit(DefaultArg)) { |