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 | 663150f6375e87b525ae1834751a47261a398b44 (patch) | |
tree | e855353f7bec2f54ffa6aaabe488e6f6d75c94f0 /llvm/lib/Target/ARM/ARMLoadStoreOptimizer.cpp | |
parent | 68e2e1b32b01fbd161c7baa4cc25fbe330431354 (diff) | |
download | llvm-663150f6375e87b525ae1834751a47261a398b44.zip llvm-663150f6375e87b525ae1834751a47261a398b44.tar.gz llvm-663150f6375e87b525ae1834751a47261a398b44.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>
llvm-svn: 184492
Diffstat (limited to 'llvm/lib/Target/ARM/ARMLoadStoreOptimizer.cpp')
-rw-r--r-- | llvm/lib/Target/ARM/ARMLoadStoreOptimizer.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/llvm/lib/Target/ARM/ARMLoadStoreOptimizer.cpp b/llvm/lib/Target/ARM/ARMLoadStoreOptimizer.cpp index c8ed576..4e97dda 100644 --- a/llvm/lib/Target/ARM/ARMLoadStoreOptimizer.cpp +++ b/llvm/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; |