6502/llvm-6502
1
0
Fork 0
mirror of https://github.com/c64scene-ar/llvm-6502.git synced 2024-12-14 11:32:34 +00:00
Code Issues Projects Releases Wiki Activity

It is safe to speculate load from GOT. This fixes performance regression caused by r141689.

Browse Source 
Radar 10281206.


git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@142202 91177308-0d34-0410-b5e6-96231b3b80d8
This commit is contained in:
Devang Patel 2011-10-17 17:35:01 +00:00
parent 4007529d4b
commit 6c15fec3c5
1 changed files with 17 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
lib/CodeGen/MachineLICM.cpp
View File

@ -762,6 +762,21 @@ void MachineLICM::UpdateRegPressure(const MachineInstr *MI) {
}
}
/// isLoadFromGOT - Return true if this machine instruction loads from
/// global offset table.
static bool isLoadFromGOT(MachineInstr &MI) {
assert (MI.getDesc().mayLoad() && "Expected MI that loads!");
for (MachineInstr::mmo_iterator I = MI.memoperands_begin(),
E = MI.memoperands_end(); I != E; ++I) {
if (const Value *V = (*I)->getValue()) {
if (const PseudoSourceValue *PSV = dyn_cast<PseudoSourceValue>(V))
if (PSV == PSV->getGOT())
return true;
}
}
return false;
}
/// IsLICMCandidate - Returns true if the instruction may be a suitable
/// candidate for LICM. e.g. If the instruction is a call, then it's obviously
/// not safe to hoist it.
@ -775,7 +790,8 @@ bool MachineLICM::IsLICMCandidate(MachineInstr &I) {
// it dominates all exiting blocks. If it doesn't, then there is a path out of
// the loop which does not execute this load, so we can't hoist it.
// Stores and side effects are already checked by isSafeToMove.
if (I.getDesc().mayLoad() && !IsGuaranteedToExecute(I.getParent()))
if (I.getDesc().mayLoad() && !isLoadFromGOT(I) &&
!IsGuaranteedToExecute(I.getParent()))
return false;
return true;