diff options
| author | Quentin Colombet <qcolombet@apple.com> | 2013-06-20 22:51:44 +0000 |
|---|---|---|
| committer | Quentin Colombet <qcolombet@apple.com> | 2013-06-20 22:51:44 +0000 |
| commit | 2b7cdf09a142b7f3e9a0ec8c7044eaf89bc59caa (patch) | |
| tree | 98e08a0ccc51076ea3ef90748d6f404aad627126 /lib/Target | |
| parent | 846565924a6f2932efc75c249b29c3619e587bbb (diff) | |
| download | external_llvm-2b7cdf09a142b7f3e9a0ec8c7044eaf89bc59caa.zip external_llvm-2b7cdf09a142b7f3e9a0ec8c7044eaf89bc59caa.tar.gz external_llvm-2b7cdf09a142b7f3e9a0ec8c7044eaf89bc59caa.tar.bz2 | |
ARM: Remove a (false) dependency on the memoryoperand's value as we do not use
it at the moment.
This allows to form more paired loads even when stack coloring pass destroys the
memoryoperand's value.
<rdar://problem/13978317>
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@184492 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/Target')
| -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 c8ed576..4e97dda 100644 --- a/lib/Target/ARM/ARMLoadStoreOptimizer.cpp +++ b/lib/Target/ARM/ARMLoadStoreOptimizer.cpp @@ -1602,8 +1602,9 @@ ARMPreAllocLoadStoreOpt::CanFormLdStDWord(MachineInstr *Op0, MachineInstr *Op1, return false; // Make sure the base address satisfies i64 ld / st alignment requirement. + // At the moment, we ignore the memoryoperand's value. + // If we want to use AliasAnalysis, we should check it accordingly. if (!Op0->hasOneMemOperand() || - !(*Op0->memoperands_begin())->getValue() || (*Op0->memoperands_begin())->isVolatile()) return false; |