diff options
| author | Chris Lattner <sabre@nondot.org> | 2012-01-04 22:35:55 +0000 |
|---|---|---|
| committer | Chris Lattner <sabre@nondot.org> | 2012-01-04 22:35:55 +0000 |
| commit | 36bc4f41bcf46f5fa12238add36c40d14cf2b42e (patch) | |
| tree | 9156d9ff5d5603f0eb3dddfbafed1b72485c7cfe /clang | |
| parent | cc0cf909538a7b387788f3a38286cae791db79ed (diff) | |
| download | bcm5719-llvm-36bc4f41bcf46f5fa12238add36c40d14cf2b42e.tar.gz bcm5719-llvm-36bc4f41bcf46f5fa12238add36c40d14cf2b42e.zip | |
implement rdar://10639962 by keeping track of increased alignment
information even in subscripting operations.
llvm-svn: 147557
Diffstat (limited to 'clang')
| -rw-r--r-- | clang/lib/CodeGen/CGExpr.cpp | 6 | ||||
| -rw-r--r-- | clang/test/CodeGen/alignment.c | 10 |
2 files changed, 15 insertions, 1 deletions
diff --git a/clang/lib/CodeGen/CGExpr.cpp b/clang/lib/CodeGen/CGExpr.cpp index 4c23c061a70..c6ba65c03c4 100644 --- a/clang/lib/CodeGen/CGExpr.cpp +++ b/clang/lib/CodeGen/CGExpr.cpp @@ -1702,13 +1702,17 @@ LValue CodeGenFunction::EmitArraySubscriptExpr(const ArraySubscriptExpr *E) { assert(!T.isNull() && "CodeGenFunction::EmitArraySubscriptExpr(): Illegal base type"); + // Limit the alignment to that of the result type. + LValue LV; if (!ArrayAlignment.isZero()) { CharUnits Align = getContext().getTypeAlignInChars(T); ArrayAlignment = std::min(Align, ArrayAlignment); + LV = MakeAddrLValue(Address, T, ArrayAlignment); + } else { + LV = MakeNaturalAlignAddrLValue(Address, T); } - LValue LV = MakeAddrLValue(Address, T, ArrayAlignment); LV.getQuals().setAddressSpace(E->getBase()->getType().getAddressSpace()); if (getContext().getLangOptions().ObjC1 && diff --git a/clang/test/CodeGen/alignment.c b/clang/test/CodeGen/alignment.c index e1c9e9eb557..9384ec83d14 100644 --- a/clang/test/CodeGen/alignment.c +++ b/clang/test/CodeGen/alignment.c @@ -47,3 +47,13 @@ void test3(packedfloat3 *p) { // CHECK: ret void + +typedef float __attribute__((vector_size(16), aligned(64))) float4align64; + +// rdar://10639962 - Typedef alignment lost in p[]-style dereferencing +void test4(float4align64 *p) { + p[0] = (float4align64){ 3.2f, 2.3f, 0.1f, 0.0f }; +} +// CHECK: @test4( +// CHECK: store <4 x float> {{.*}}, <4 x float>* %arrayidx, align 64 + |
