From 5caa883afc2c768c293757d4ca30d85b9094e876 Mon Sep 17 00:00:00 2001 From: Evan Cheng Date: Wed, 4 Feb 2009 09:21:58 +0000 Subject: [PATCH] Don't call isInvariantLoad twice. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@63729 91177308-0d34-0410-b5e6-96231b3b80d8 --- lib/CodeGen/MachineLICM.cpp | 11 ++--------- 1 file changed, 2 insertions(+), 9 deletions(-) diff --git a/lib/CodeGen/MachineLICM.cpp b/lib/CodeGen/MachineLICM.cpp index 0a9f7207b1f..10ee8d6d137 100644 --- a/lib/CodeGen/MachineLICM.cpp +++ b/lib/CodeGen/MachineLICM.cpp @@ -276,18 +276,11 @@ static bool HasOnlyPHIUses(unsigned Reg, MachineRegisterInfo *RegInfo) { bool MachineLICM::IsProfitableToHoist(MachineInstr &MI) { const TargetInstrDesc &TID = MI.getDesc(); - bool isInvLoad = false; - if (TID.mayLoad()) { - isInvLoad = TII->isInvariantLoad(&MI); - if (!isInvLoad) - return false; - } - // FIXME: For now, only hoist re-materilizable instructions. LICM will // increase register pressure. We want to make sure it doesn't increase // spilling. - if (!isInvLoad && (!TID.isRematerializable() || - !TII->isTriviallyReMaterializable(&MI))) + if (!TID.mayLoad() && (!TID.isRematerializable() || + !TII->isTriviallyReMaterializable(&MI))) return false; if (!TID.isAsCheapAsAMove())