diff options
author | Alexey Bataev <a.bataev@hotmail.com> | 2014-12-18 06:54:53 +0000 |
---|---|---|
committer | Alexey Bataev <a.bataev@hotmail.com> | 2014-12-18 06:54:53 +0000 |
commit | 7cb17890114fa63ecdc440e1e2376726cc8fce19 (patch) | |
tree | 401b252be45c875310b09eced9975e0806c7797d /clang/lib/CodeGen | |
parent | f5ba8288ad715d59361704f48bfc0f4a6ded998b (diff) | |
download | bcm5719-llvm-7cb17890114fa63ecdc440e1e2376726cc8fce19.tar.gz bcm5719-llvm-7cb17890114fa63ecdc440e1e2376726cc8fce19.zip |
Fix for PR21915: assert on multidimensional VLA in function arguments.
Fixed assertion on type checking for arguments and parameters on function call if arguments are pointers to VLA
Differential Revision: http://reviews.llvm.org/D6655
llvm-svn: 224504
Diffstat (limited to 'clang/lib/CodeGen')
-rw-r--r-- | clang/lib/CodeGen/CodeGenFunction.h | 29 |
1 files changed, 7 insertions, 22 deletions
diff --git a/clang/lib/CodeGen/CodeGenFunction.h b/clang/lib/CodeGen/CodeGenFunction.h index 2e1e400041a..e2fd80e7d3e 100644 --- a/clang/lib/CodeGen/CodeGenFunction.h +++ b/clang/lib/CodeGen/CodeGenFunction.h @@ -2746,28 +2746,13 @@ public: E = CallArgTypeInfo->param_type_end(); I != E; ++I, ++Arg) { assert(Arg != ArgEnd && "Running over edge of argument list!"); -#ifndef NDEBUG - QualType ArgType = *I; - QualType ActualArgType = Arg->getType(); - if (ArgType->isPointerType() && ActualArgType->isPointerType()) { - QualType ActualBaseType = - ActualArgType->getAs<PointerType>()->getPointeeType(); - QualType ArgBaseType = - ArgType->getAs<PointerType>()->getPointeeType(); - if (ArgBaseType->isVariableArrayType()) { - if (const VariableArrayType *VAT = - getContext().getAsVariableArrayType(ActualBaseType)) { - if (!VAT->getSizeExpr()) - ActualArgType = ArgType; - } - } - } - assert(getContext() - .getCanonicalType(ArgType.getNonReferenceType()) - .getTypePtr() == - getContext().getCanonicalType(ActualArgType).getTypePtr() && - "type mismatch in call argument!"); -#endif + assert( + ((*I)->isVariablyModifiedType() || + getContext() + .getCanonicalType((*I).getNonReferenceType()) + .getTypePtr() == + getContext().getCanonicalType(Arg->getType()).getTypePtr()) && + "type mismatch in call argument!"); ArgTypes.push_back(*I); } } |