diff options
author | Florian Hahn <florian.hahn@arm.com> | 2017-06-21 09:51:52 +0000 |
---|---|---|
committer | Florian Hahn <florian.hahn@arm.com> | 2017-06-21 09:51:52 +0000 |
commit | 8552e591a12f41a142b1214f9081c8d88db46337 (patch) | |
tree | 1630dc40fe5a836c5104a8c9f8df5a30ab641598 /llvm/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp | |
parent | d280ea4f763786cfe037af6bc7baec0614017e97 (diff) | |
download | bcm5719-llvm-8552e591a12f41a142b1214f9081c8d88db46337.tar.gz bcm5719-llvm-8552e591a12f41a142b1214f9081c8d88db46337.zip |
[AArch64] Add early exit to promoteLoadFromStore.
There should be at most a single kill flag for the
promoted operand between the store/load pair.
Discussed in https://reviews.llvm.org/D34402.
llvm-svn: 305889
Diffstat (limited to 'llvm/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp')
-rw-r--r-- | llvm/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/llvm/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp b/llvm/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp index c35de77727e..c45c0b4c8ed 100644 --- a/llvm/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp +++ b/llvm/lib/Target/AArch64/AArch64LoadStoreOptimizer.cpp @@ -877,7 +877,10 @@ AArch64LoadStoreOpt::promoteLoadFromStore(MachineBasicBlock::iterator LoadI, // Clear kill flags between store and load. for (MachineInstr &MI : make_range(StoreI->getIterator(), BitExtMI->getIterator())) - MI.clearRegisterKills(StRt, TRI); + if (MI.killsRegister(StRt, TRI)) { + MI.clearRegisterKills(StRt, TRI); + break; + } DEBUG(dbgs() << "Promoting load by replacing :\n "); DEBUG(StoreI->print(dbgs())); |