diff options
| author | Daniel Sanders <daniel_l_sanders@apple.com> | 2018-05-09 05:00:17 +0000 |
|---|---|---|
| committer | Daniel Sanders <daniel_l_sanders@apple.com> | 2018-05-09 05:00:17 +0000 |
| commit | 618437459cd03ab47bcab7b1ba6fd1736f4f78c7 (patch) | |
| tree | d4cd10ac1ccd018a8a8b38f146df64b95ac8f41f /llvm/lib/CodeGen | |
| parent | 801fca259ed90d9cebcd0c90ac428198d07401f5 (diff) | |
| download | bcm5719-llvm-618437459cd03ab47bcab7b1ba6fd1736f4f78c7.tar.gz bcm5719-llvm-618437459cd03ab47bcab7b1ba6fd1736f4f78c7.zip | |
Revert r331816 and r331820 - [globalisel] Add a combiner helpers for extending loads and use them in a pre-legalize combiner for AArch64
Reverting this to see if the clang-cmake-aarch64-global-isel and
clang-cmake-aarch64-quick bots are failing because of this commit.
We know it wasn't r331819.
llvm-svn: 331846
Diffstat (limited to 'llvm/lib/CodeGen')
| -rw-r--r-- | llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp | 34 |
1 files changed, 1 insertions, 33 deletions
diff --git a/llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp b/llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp index da797ee1b25..44e904a6391 100644 --- a/llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp +++ b/llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp @@ -36,38 +36,6 @@ bool CombinerHelper::tryCombineCopy(MachineInstr &MI) { return false; } -bool CombinerHelper::tryCombineExtendingLoads(MachineInstr &MI) { - if (MI.getOpcode() != TargetOpcode::G_ANYEXT && - MI.getOpcode() != TargetOpcode::G_SEXT && - MI.getOpcode() != TargetOpcode::G_ZEXT) - return false; - - unsigned DstReg = MI.getOperand(0).getReg(); - unsigned SrcReg = MI.getOperand(1).getReg(); - - LLT DstTy = MRI.getType(DstReg); - if (!DstTy.isScalar()) - return false; - - if (MachineInstr *DefMI = getOpcodeDef(TargetOpcode::G_LOAD, SrcReg, MRI)) { - unsigned PtrReg = DefMI->getOperand(1).getReg(); - MachineMemOperand &MMO = **DefMI->memoperands_begin(); - DEBUG(dbgs() << ".. Combine MI: " << MI;); - Builder.setInstr(MI); - Builder.buildLoadInstr(MI.getOpcode() == TargetOpcode::G_SEXT - ? TargetOpcode::G_SEXTLOAD - : MI.getOpcode() == TargetOpcode::G_ZEXT - ? TargetOpcode::G_ZEXTLOAD - : TargetOpcode::G_LOAD, - DstReg, PtrReg, MMO); - MI.eraseFromParent(); - return true; - } - return false; -} - bool CombinerHelper::tryCombine(MachineInstr &MI) { - if (tryCombineCopy(MI)) - return true; - return tryCombineExtendingLoads(MI); + return tryCombineCopy(MI); } |

