diff options
author | Anders Carlsson <andersca@mac.com> | 2008-12-21 03:33:21 +0000 |
---|---|---|
committer | Anders Carlsson <andersca@mac.com> | 2008-12-21 03:33:21 +0000 |
commit | b50525ba0e996bc072cdb76152fcfe0bc64bb72a (patch) | |
tree | 8534b716aa047385dcd3e2e1be544e60e2ee8055 | |
parent | 56a3460cf2ce104a88f6642365721f4a623e9cd5 (diff) |
Make sure to emit the size expression for sizeof(type)
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@61301 91177308-0d34-0410-b5e6-96231b3b80d8
-rw-r--r-- | lib/CodeGen/CGExprScalar.cpp | 7 | ||||
-rw-r--r-- | test/CodeGen/vla.c | 5 |
2 files changed, 11 insertions, 1 deletions
diff --git a/lib/CodeGen/CGExprScalar.cpp b/lib/CodeGen/CGExprScalar.cpp index 4984d0df35..d829facb16 100644 --- a/lib/CodeGen/CGExprScalar.cpp +++ b/lib/CodeGen/CGExprScalar.cpp @@ -663,8 +663,13 @@ ScalarExprEmitter::VisitSizeOfAlignOfExpr(const SizeOfAlignOfExpr *E) { if (const VariableArrayType *VAT = CGF.getContext().getAsVariableArrayType(TypeToSize)) { - if (E->isSizeOf()) + if (E->isSizeOf()) { + if (E->isArgumentType()) { + // sizeof(type) - make sure to emit the VLA size. + CGF.EmitVLASize(TypeToSize); + } return CGF.GetVLASize(VAT); + } // FIXME: This should be an UNSUPPORTED error. assert(0 && "alignof VLAs not implemented yet"); } diff --git a/test/CodeGen/vla.c b/test/CodeGen/vla.c index a584020abc..0881520c75 100644 --- a/test/CodeGen/vla.c +++ b/test/CodeGen/vla.c @@ -8,3 +8,8 @@ void a(int x) { arry[0] = 10; b(arry); } + +void b(int n) +{ + sizeof(int[n]); +} |