diff options
| author | Bob Wilson <bob.wilson@apple.com> | 2010-09-10 05:15:04 +0000 |
|---|---|---|
| committer | Bob Wilson <bob.wilson@apple.com> | 2010-09-10 05:15:04 +0000 |
| commit | bd9c766fa67f718f225e461ed69c3d11cfe7068c (patch) | |
| tree | 9023ab09a14db518e7347f48b47925049acd1471 /lib/Target/ARM/ARMLoadStoreOptimizer.cpp | |
| parent | 0cf9a820e435a74ef93930dcaf8917323f2f99be (diff) | |
| download | external_llvm-bd9c766fa67f718f225e461ed69c3d11cfe7068c.zip external_llvm-bd9c766fa67f718f225e461ed69c3d11cfe7068c.tar.gz external_llvm-bd9c766fa67f718f225e461ed69c3d11cfe7068c.tar.bz2 | |
Fix merging base-updates for VLDM/VSTM: Before I switched these instructions
to use AddrMode4, there was a count of the registers stored in one of the
operands. I changed that to just count the operands but forgot to adjust for
the size of D registers. This was noticed by Evan as a performance problem
but it is a potential correctness bug as well, since it is possible that this
could merge a base update with a non-matching immediate.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@113576 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target/ARM/ARMLoadStoreOptimizer.cpp')
| -rw-r--r-- | lib/Target/ARM/ARMLoadStoreOptimizer.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/lib/Target/ARM/ARMLoadStoreOptimizer.cpp b/lib/Target/ARM/ARMLoadStoreOptimizer.cpp index d1acacf..2b7645a 100644 --- a/lib/Target/ARM/ARMLoadStoreOptimizer.cpp +++ b/lib/Target/ARM/ARMLoadStoreOptimizer.cpp @@ -458,9 +458,10 @@ static inline unsigned getLSMultipleTransferSize(MachineInstr *MI) { case ARM::t2STM: case ARM::VLDMS: case ARM::VSTMS: + return (MI->getNumOperands() - 4) * 4; case ARM::VLDMD: case ARM::VSTMD: - return (MI->getNumOperands() - 4) * 4; + return (MI->getNumOperands() - 4) * 8; } } |