2012-02-18 20:03:15 +08:00
|
|
|
//===-- HexagonCFGOptimizer.cpp - CFG optimizations -----------------------===//
|
2011-12-13 05:14:40 +08:00
|
|
|
// The LLVM Compiler Infrastructure
|
|
|
|
//
|
|
|
|
// This file is distributed under the University of Illinois Open Source
|
|
|
|
// License. See LICENSE.TXT for details.
|
|
|
|
//
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
2012-12-04 00:50:05 +08:00
|
|
|
#include "Hexagon.h"
|
2012-02-06 18:19:29 +08:00
|
|
|
#include "HexagonMachineFunctionInfo.h"
|
2012-12-04 00:50:05 +08:00
|
|
|
#include "HexagonSubtarget.h"
|
|
|
|
#include "HexagonTargetMachine.h"
|
2011-12-13 05:14:40 +08:00
|
|
|
#include "llvm/CodeGen/MachineDominators.h"
|
|
|
|
#include "llvm/CodeGen/MachineFunctionPass.h"
|
2012-02-06 18:19:29 +08:00
|
|
|
#include "llvm/CodeGen/MachineInstrBuilder.h"
|
2011-12-13 05:14:40 +08:00
|
|
|
#include "llvm/CodeGen/MachineLoopInfo.h"
|
|
|
|
#include "llvm/CodeGen/MachineRegisterInfo.h"
|
2012-02-06 18:19:29 +08:00
|
|
|
#include "llvm/CodeGen/Passes.h"
|
2011-12-13 05:14:40 +08:00
|
|
|
#include "llvm/Support/Compiler.h"
|
|
|
|
#include "llvm/Support/Debug.h"
|
|
|
|
#include "llvm/Support/MathExtras.h"
|
2012-12-04 00:50:05 +08:00
|
|
|
#include "llvm/Target/TargetInstrInfo.h"
|
|
|
|
#include "llvm/Target/TargetMachine.h"
|
|
|
|
#include "llvm/Target/TargetRegisterInfo.h"
|
2011-12-13 05:14:40 +08:00
|
|
|
|
|
|
|
using namespace llvm;
|
|
|
|
|
2014-04-22 10:41:26 +08:00
|
|
|
#define DEBUG_TYPE "hexagon_cfg"
|
|
|
|
|
2013-05-07 05:58:00 +08:00
|
|
|
namespace llvm {
|
2015-06-16 03:05:35 +08:00
|
|
|
FunctionPass *createHexagonCFGOptimizer();
|
2013-05-07 05:58:00 +08:00
|
|
|
void initializeHexagonCFGOptimizerPass(PassRegistry&);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
2011-12-13 05:14:40 +08:00
|
|
|
namespace {
|
|
|
|
|
|
|
|
class HexagonCFGOptimizer : public MachineFunctionPass {
|
|
|
|
|
|
|
|
private:
|
|
|
|
void InvertAndChangeJumpTarget(MachineInstr*, MachineBasicBlock*);
|
|
|
|
|
|
|
|
public:
|
|
|
|
static char ID;
|
2015-02-03 02:46:27 +08:00
|
|
|
HexagonCFGOptimizer() : MachineFunctionPass(ID) {
|
2013-05-07 05:58:00 +08:00
|
|
|
initializeHexagonCFGOptimizerPass(*PassRegistry::getPassRegistry());
|
|
|
|
}
|
2011-12-13 05:14:40 +08:00
|
|
|
|
2014-04-29 15:58:16 +08:00
|
|
|
const char *getPassName() const override {
|
2011-12-13 05:14:40 +08:00
|
|
|
return "Hexagon CFG Optimizer";
|
|
|
|
}
|
2014-04-29 15:58:16 +08:00
|
|
|
bool runOnMachineFunction(MachineFunction &Fn) override;
|
2011-12-13 05:14:40 +08:00
|
|
|
};
|
|
|
|
|
|
|
|
|
|
|
|
char HexagonCFGOptimizer::ID = 0;
|
|
|
|
|
|
|
|
static bool IsConditionalBranch(int Opc) {
|
2014-12-11 05:24:10 +08:00
|
|
|
return (Opc == Hexagon::J2_jumpt) || (Opc == Hexagon::J2_jumpf)
|
|
|
|
|| (Opc == Hexagon::J2_jumptnewpt) || (Opc == Hexagon::J2_jumpfnewpt);
|
2011-12-13 05:14:40 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
static bool IsUnconditionalJump(int Opc) {
|
2014-12-11 05:24:10 +08:00
|
|
|
return (Opc == Hexagon::J2_jump);
|
2011-12-13 05:14:40 +08:00
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
void
|
|
|
|
HexagonCFGOptimizer::InvertAndChangeJumpTarget(MachineInstr* MI,
|
|
|
|
MachineBasicBlock* NewTarget) {
|
2015-02-03 02:46:27 +08:00
|
|
|
const TargetInstrInfo *TII =
|
|
|
|
MI->getParent()->getParent()->getSubtarget().getInstrInfo();
|
2011-12-13 05:14:40 +08:00
|
|
|
int NewOpcode = 0;
|
|
|
|
switch(MI->getOpcode()) {
|
2014-12-11 05:24:10 +08:00
|
|
|
case Hexagon::J2_jumpt:
|
|
|
|
NewOpcode = Hexagon::J2_jumpf;
|
2011-12-13 05:14:40 +08:00
|
|
|
break;
|
|
|
|
|
2014-12-11 05:24:10 +08:00
|
|
|
case Hexagon::J2_jumpf:
|
|
|
|
NewOpcode = Hexagon::J2_jumpt;
|
2011-12-13 05:14:40 +08:00
|
|
|
break;
|
|
|
|
|
2014-12-11 05:24:10 +08:00
|
|
|
case Hexagon::J2_jumptnewpt:
|
|
|
|
NewOpcode = Hexagon::J2_jumpfnewpt;
|
2011-12-13 05:14:40 +08:00
|
|
|
break;
|
|
|
|
|
2014-12-11 05:24:10 +08:00
|
|
|
case Hexagon::J2_jumpfnewpt:
|
|
|
|
NewOpcode = Hexagon::J2_jumptnewpt;
|
2011-12-13 05:14:40 +08:00
|
|
|
break;
|
|
|
|
|
|
|
|
default:
|
2012-02-07 10:50:20 +08:00
|
|
|
llvm_unreachable("Cannot handle this case");
|
2011-12-13 05:14:40 +08:00
|
|
|
}
|
|
|
|
|
2015-02-03 02:46:27 +08:00
|
|
|
MI->setDesc(TII->get(NewOpcode));
|
2011-12-13 05:14:40 +08:00
|
|
|
MI->getOperand(1).setMBB(NewTarget);
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
|
|
bool HexagonCFGOptimizer::runOnMachineFunction(MachineFunction &Fn) {
|
|
|
|
// Loop over all of the basic blocks.
|
|
|
|
for (MachineFunction::iterator MBBb = Fn.begin(), MBBe = Fn.end();
|
|
|
|
MBBb != MBBe; ++MBBb) {
|
Hexagon: Remove implicit ilist iterator conversions, NFC
There are two things out of the ordinary in this commit. First, I made
a loop obviously "infinite" in HexagonInstrInfo.cpp. After checking if
an instruction was at the beginning of a basic block (in which case,
`break`), the loop decremented and checked the iterator for `nullptr` as
the loop condition. This has never been possible (the prev pointers are
always been circular, so even with the weird ilist/iplist
implementation, this isn't been possible), so I removed the condition.
Second, in HexagonAsmPrinter.cpp there was another case of comparing a
`MachineBasicBlock::instr_iterator` against `MachineBasicBlock::end()`
(which returns `MachineBasicBlock::iterator`). While not incorrect,
it's fragile. I switched this to `::instr_end()`.
All that said, no functionality change intended here.
llvm-svn: 250778
2015-10-20 08:46:39 +08:00
|
|
|
MachineBasicBlock *MBB = &*MBBb;
|
2011-12-13 05:14:40 +08:00
|
|
|
|
|
|
|
// Traverse the basic block.
|
|
|
|
MachineBasicBlock::iterator MII = MBB->getFirstTerminator();
|
|
|
|
if (MII != MBB->end()) {
|
|
|
|
MachineInstr *MI = MII;
|
|
|
|
int Opc = MI->getOpcode();
|
|
|
|
if (IsConditionalBranch(Opc)) {
|
|
|
|
|
|
|
|
//
|
|
|
|
// (Case 1) Transform the code if the following condition occurs:
|
|
|
|
// BB1: if (p0) jump BB3
|
|
|
|
// ...falls-through to BB2 ...
|
|
|
|
// BB2: jump BB4
|
|
|
|
// ...next block in layout is BB3...
|
|
|
|
// BB3: ...
|
|
|
|
//
|
|
|
|
// Transform this to:
|
|
|
|
// BB1: if (!p0) jump BB4
|
|
|
|
// Remove BB2
|
|
|
|
// BB3: ...
|
|
|
|
//
|
|
|
|
// (Case 2) A variation occurs when BB3 contains a JMP to BB4:
|
|
|
|
// BB1: if (p0) jump BB3
|
|
|
|
// ...falls-through to BB2 ...
|
|
|
|
// BB2: jump BB4
|
|
|
|
// ...other basic blocks ...
|
|
|
|
// BB4:
|
|
|
|
// ...not a fall-thru
|
|
|
|
// BB3: ...
|
|
|
|
// jump BB4
|
|
|
|
//
|
|
|
|
// Transform this to:
|
|
|
|
// BB1: if (!p0) jump BB4
|
|
|
|
// Remove BB2
|
|
|
|
// BB3: ...
|
|
|
|
// BB4: ...
|
|
|
|
//
|
|
|
|
unsigned NumSuccs = MBB->succ_size();
|
|
|
|
MachineBasicBlock::succ_iterator SI = MBB->succ_begin();
|
|
|
|
MachineBasicBlock* FirstSucc = *SI;
|
|
|
|
MachineBasicBlock* SecondSucc = *(++SI);
|
2014-04-25 13:30:21 +08:00
|
|
|
MachineBasicBlock* LayoutSucc = nullptr;
|
|
|
|
MachineBasicBlock* JumpAroundTarget = nullptr;
|
2011-12-13 05:14:40 +08:00
|
|
|
|
|
|
|
if (MBB->isLayoutSuccessor(FirstSucc)) {
|
|
|
|
LayoutSucc = FirstSucc;
|
|
|
|
JumpAroundTarget = SecondSucc;
|
|
|
|
} else if (MBB->isLayoutSuccessor(SecondSucc)) {
|
|
|
|
LayoutSucc = SecondSucc;
|
|
|
|
JumpAroundTarget = FirstSucc;
|
|
|
|
} else {
|
|
|
|
// Odd case...cannot handle.
|
|
|
|
}
|
|
|
|
|
|
|
|
// The target of the unconditional branch must be JumpAroundTarget.
|
|
|
|
// TODO: If not, we should not invert the unconditional branch.
|
2014-04-25 13:30:21 +08:00
|
|
|
MachineBasicBlock* CondBranchTarget = nullptr;
|
2014-12-11 05:24:10 +08:00
|
|
|
if ((MI->getOpcode() == Hexagon::J2_jumpt) ||
|
|
|
|
(MI->getOpcode() == Hexagon::J2_jumpf)) {
|
2011-12-13 05:14:40 +08:00
|
|
|
CondBranchTarget = MI->getOperand(1).getMBB();
|
|
|
|
}
|
|
|
|
|
|
|
|
if (!LayoutSucc || (CondBranchTarget != JumpAroundTarget)) {
|
|
|
|
continue;
|
|
|
|
}
|
|
|
|
|
|
|
|
if ((NumSuccs == 2) && LayoutSucc && (LayoutSucc->pred_size() == 1)) {
|
|
|
|
|
|
|
|
// Ensure that BB2 has one instruction -- an unconditional jump.
|
|
|
|
if ((LayoutSucc->size() == 1) &&
|
|
|
|
IsUnconditionalJump(LayoutSucc->front().getOpcode())) {
|
|
|
|
MachineBasicBlock* UncondTarget =
|
|
|
|
LayoutSucc->front().getOperand(0).getMBB();
|
|
|
|
// Check if the layout successor of BB2 is BB3.
|
|
|
|
bool case1 = LayoutSucc->isLayoutSuccessor(JumpAroundTarget);
|
|
|
|
bool case2 = JumpAroundTarget->isSuccessor(UncondTarget) &&
|
|
|
|
JumpAroundTarget->size() >= 1 &&
|
|
|
|
IsUnconditionalJump(JumpAroundTarget->back().getOpcode()) &&
|
|
|
|
JumpAroundTarget->pred_size() == 1 &&
|
|
|
|
JumpAroundTarget->succ_size() == 1;
|
|
|
|
|
|
|
|
if (case1 || case2) {
|
|
|
|
InvertAndChangeJumpTarget(MI, UncondTarget);
|
2015-12-01 13:29:22 +08:00
|
|
|
MBB->replaceSuccessor(JumpAroundTarget, UncondTarget);
|
2011-12-13 05:14:40 +08:00
|
|
|
|
|
|
|
// Remove the unconditional branch in LayoutSucc.
|
|
|
|
LayoutSucc->erase(LayoutSucc->begin());
|
2015-12-01 13:29:22 +08:00
|
|
|
LayoutSucc->replaceSuccessor(UncondTarget, JumpAroundTarget);
|
2011-12-13 05:14:40 +08:00
|
|
|
|
|
|
|
// This code performs the conversion for case 2, which moves
|
|
|
|
// the block to the fall-thru case (BB3 in the code above).
|
|
|
|
if (case2 && !case1) {
|
|
|
|
JumpAroundTarget->moveAfter(LayoutSucc);
|
|
|
|
// only move a block if it doesn't have a fall-thru. otherwise
|
|
|
|
// the CFG will be incorrect.
|
|
|
|
if (!UncondTarget->canFallThrough()) {
|
|
|
|
UncondTarget->moveAfter(JumpAroundTarget);
|
|
|
|
}
|
|
|
|
}
|
|
|
|
|
|
|
|
//
|
|
|
|
// Correct live-in information. Is used by post-RA scheduler
|
|
|
|
// The live-in to LayoutSucc is now all values live-in to
|
|
|
|
// JumpAroundTarget.
|
|
|
|
//
|
2015-09-10 02:08:03 +08:00
|
|
|
std::vector<MachineBasicBlock::RegisterMaskPair> OrigLiveIn(
|
|
|
|
LayoutSucc->livein_begin(), LayoutSucc->livein_end());
|
|
|
|
std::vector<MachineBasicBlock::RegisterMaskPair> NewLiveIn(
|
|
|
|
JumpAroundTarget->livein_begin(),
|
|
|
|
JumpAroundTarget->livein_end());
|
|
|
|
for (const auto &OrigLI : OrigLiveIn)
|
|
|
|
LayoutSucc->removeLiveIn(OrigLI.PhysReg);
|
|
|
|
for (const auto &NewLI : NewLiveIn)
|
|
|
|
LayoutSucc->addLiveIn(NewLI);
|
2011-12-13 05:14:40 +08:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|
|
|
|
return true;
|
|
|
|
}
|
2015-06-23 17:49:53 +08:00
|
|
|
}
|
2011-12-13 05:14:40 +08:00
|
|
|
|
|
|
|
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
// Public Constructor Functions
|
|
|
|
//===----------------------------------------------------------------------===//
|
|
|
|
|
2013-05-07 05:58:00 +08:00
|
|
|
static void initializePassOnce(PassRegistry &Registry) {
|
|
|
|
PassInfo *PI = new PassInfo("Hexagon CFG Optimizer", "hexagon-cfg",
|
2014-04-25 13:30:21 +08:00
|
|
|
&HexagonCFGOptimizer::ID, nullptr, false, false);
|
2013-05-07 05:58:00 +08:00
|
|
|
Registry.registerPass(*PI, true);
|
|
|
|
}
|
|
|
|
|
|
|
|
void llvm::initializeHexagonCFGOptimizerPass(PassRegistry &Registry) {
|
|
|
|
CALL_ONCE_INITIALIZATION(initializePassOnce)
|
|
|
|
}
|
|
|
|
|
2015-02-03 02:46:27 +08:00
|
|
|
FunctionPass *llvm::createHexagonCFGOptimizer() {
|
|
|
|
return new HexagonCFGOptimizer();
|
2011-12-13 05:14:40 +08:00
|
|
|
}
|