maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

Fix <rdar://problem/5733511> clang doesn't emit error for const array. 

llvm-svn: 46905
This commit is contained in:
Steve Naroff 2008-02-09 16:59:44 +00:00
parent a62f3e1352
commit 45173e0ac0
2 changed files with 19 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
clang/Sema/SemaExpr.cpp
View File

@ -904,8 +904,17 @@ void Sema::DefaultFunctionArrayConversion(Expr *&e) {
}
if (t->isFunctionType())
ImpCastExprToType(e, Context.getPointerType(t));
else if (const ArrayType *ary = t->getAsArrayType())
ImpCastExprToType(e, Context.getPointerType(ary->getElementType()));
else if (const ArrayType *ary = t->getAsArrayType()) {
// Make sure we don't loose qualifiers when dealing with typedefs. Example:
// typedef int arr[10];
// void test2() {
// const arr b;
// b[4] = 1;
// }
QualType ELT = ary->getElementType();
ELT = ELT.getQualifiedType(t.getQualifiers()|ELT.getQualifiers());
ImpCastExprToType(e, Context.getPointerType(ELT));
}
}
/// UsualUnaryConversions - Performs various conversions that are common to most

8
clang/test/Sema/assign.c
View File

@ -5,3 +5,11 @@ void *test1(void) { return 0; }
void test2 (const struct {int a;} *x) {
x->a = 10; // expected-error {{read-only variable is not assignable}}
}
typedef int arr[10];
void test3() {
const arr b;
const int b2[10];
b[4] = 1; // expected-error {{read-only variable is not assignable}}
b2[4] = 1; // expected-error {{read-only variable is not assignable}}
}