diff options
author | Chad Rosier <mcrosier@apple.com> | 2011-12-06 01:44:17 +0000 |
---|---|---|
committer | Chad Rosier <mcrosier@apple.com> | 2011-12-06 01:44:17 +0000 |
commit | c77830d21e6403b42d9511bd59a4892441f10d65 (patch) | |
tree | 4c67cb8b6c2fea9bcfc689032815aeb6b021ecbc | |
parent | 2e05db2fa06e90298b428c91e9a4c8c1e367a7b9 (diff) | |
download | bcm5719-llvm-c77830d21e6403b42d9511bd59a4892441f10d65.tar.gz bcm5719-llvm-c77830d21e6403b42d9511bd59a4892441f10d65.zip |
[arm-fast-isel] Doublewords only require word-alignment.
rdar://10528060
llvm-svn: 145891
-rw-r--r-- | llvm/lib/Target/ARM/ARMFastISel.cpp | 7 | ||||
-rw-r--r-- | llvm/test/CodeGen/ARM/fast-isel.ll | 18 |
2 files changed, 22 insertions, 3 deletions
diff --git a/llvm/lib/Target/ARM/ARMFastISel.cpp b/llvm/lib/Target/ARM/ARMFastISel.cpp index ea03fb7a601..de366f7bc8f 100644 --- a/llvm/lib/Target/ARM/ARMFastISel.cpp +++ b/llvm/lib/Target/ARM/ARMFastISel.cpp @@ -1106,7 +1106,7 @@ bool ARMFastISel::ARMEmitStore(EVT VT, unsigned SrcReg, Address &Addr, case MVT::f32: if (!Subtarget->hasVFP2()) return false; StrOpc = ARM::VSTRS; - // Unaligned stores need special handling. + // Unaligned stores need special handling. Floats require word-alignment. if (Alignment && Alignment < 4) { unsigned MoveReg = createResultReg(TLI.getRegClassFor(MVT::i32)); AddOptionalDefs(BuildMI(*FuncInfo.MBB, FuncInfo.InsertPt, DL, @@ -1119,8 +1119,9 @@ bool ARMFastISel::ARMEmitStore(EVT VT, unsigned SrcReg, Address &Addr, break; case MVT::f64: if (!Subtarget->hasVFP2()) return false; - // FIXME: Unaligned stores need special handling. - if (Alignment && Alignment < 8) { + // FIXME: Unaligned stores need special handling. Doublewords require + // word-alignment. + if (Alignment && Alignment < 4) { return false; } StrOpc = ARM::VSTRD; diff --git a/llvm/test/CodeGen/ARM/fast-isel.ll b/llvm/test/CodeGen/ARM/fast-isel.ll index f9c1a9d7d35..06974608738 100644 --- a/llvm/test/CodeGen/ARM/fast-isel.ll +++ b/llvm/test/CodeGen/ARM/fast-isel.ll @@ -180,3 +180,21 @@ entry: store float %add, float* %x1, align 1 ret void } + +; Doublewords require only word-alignment. +; rdar://10528060 +%struct.anon.0 = type { double } + +@foo_unpacked = common global %struct.anon.0 zeroinitializer, align 4 + +define void @test5(double %a, double %b) nounwind { +entry: +; ARM: @test5 +; THUMB: @test5 + %add = fadd double %a, %b + store double %add, double* getelementptr inbounds (%struct.anon.0* @foo_unpacked, i32 0, i32 0), align 4 +; ARM: vstr d16, [r0] +; THUMB: vstr d16, [r0] + ret void +} + |