mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2024-10-22 12:33:33 +02:00
[Hexagon] Avoid register dependencies on indirect branches in packetizer
Do not packetize the instruction setting the branch address with the indirect branch itself. llvm-svn: 279324
This commit is contained in:
parent
3486149e3b
commit
0c8510db02
@ -283,11 +283,18 @@ bool HexagonPacketizerList::isCallDependent(const MachineInstr &MI,
|
||||
|
||||
// Assumes that the first operand of the CALLr is the function address.
|
||||
if (HII->isIndirectCall(MI) && (DepType == SDep::Data)) {
|
||||
MachineOperand MO = MI.getOperand(0);
|
||||
const MachineOperand MO = MI.getOperand(0);
|
||||
if (MO.isReg() && MO.isUse() && (MO.getReg() == DepReg))
|
||||
return true;
|
||||
}
|
||||
|
||||
if (HII->isJumpR(MI)) {
|
||||
const MachineOperand &MO = HII->isPredicated(MI) ? MI.getOperand(1)
|
||||
: MI.getOperand(0);
|
||||
assert(MO.isReg() && MO.isUse());
|
||||
if (MO.getReg() == DepReg)
|
||||
return true;
|
||||
}
|
||||
return false;
|
||||
}
|
||||
|
||||
@ -1293,12 +1300,6 @@ bool HexagonPacketizerList::isLegalToPacketizeTogether(SUnit *SUI, SUnit *SUJ) {
|
||||
// dealloc return unless we have dependencies on the explicit uses
|
||||
// of the registers used by jumpr (like r31) or dealloc return
|
||||
// (like r29 or r30).
|
||||
//
|
||||
// TODO: Currently, jumpr is handling only return of r31. So, the
|
||||
// following logic (specificaly isCallDependent) is working fine.
|
||||
// We need to enable jumpr for register other than r31 and then,
|
||||
// we need to rework the last part, where it handles indirect call
|
||||
// of that (isCallDependent) function. Bug 6216 is opened for this.
|
||||
unsigned DepReg = 0;
|
||||
const TargetRegisterClass *RC = nullptr;
|
||||
if (DepType == SDep::Data) {
|
||||
|
Loading…
Reference in New Issue
Block a user