mirror of
https://github.com/RPCS3/llvm-mirror.git
synced 2025-02-01 13:11:39 +01:00
Refactor code that checks if it's a call to a "nounwind" function.
llvm-svn: 87036
This commit is contained in:
parent
9a1f2aaff0
commit
507f0b33a6
@ -462,6 +462,40 @@ ComputeActionsTable(const SmallVectorImpl<const LandingPadInfo*> &LandingPads,
|
||||
return SizeActions;
|
||||
}
|
||||
|
||||
/// CallToNoUnwindFunction - Return `true' if this is a call to a function
|
||||
/// marked `nounwind'. Return `false' otherwise.
|
||||
bool DwarfException::CallToNoUnwindFunction(const MachineInstr *MI) {
|
||||
assert(MI->getDesc().isCall() && "This should be a call instruction!");
|
||||
|
||||
bool MarkedNoUnwind = false;
|
||||
bool SawFunc = false;
|
||||
|
||||
for (unsigned I = 0, E = MI->getNumOperands(); I != E; ++I) {
|
||||
const MachineOperand &MO = MI->getOperand(I);
|
||||
|
||||
if (MO.isGlobal()) {
|
||||
if (Function *F = dyn_cast<Function>(MO.getGlobal())) {
|
||||
if (SawFunc) {
|
||||
// Be conservative. If we have more than one function operand for this
|
||||
// call, then we can't make the assumption that it's the callee and
|
||||
// not a parameter to the call.
|
||||
//
|
||||
// FIXME: Determine if there's a way to say that `F' is the callee or
|
||||
// parameter.
|
||||
MarkedNoUnwind = false;
|
||||
break;
|
||||
}
|
||||
if (F->doesNotThrow()) {
|
||||
SawFunc = true;
|
||||
MarkedNoUnwind = true;
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
return MarkedNoUnwind;
|
||||
}
|
||||
|
||||
/// ComputeCallSiteTable - Compute the call-site table. The entry for an invoke
|
||||
/// has a try-range containing the call, a non-zero landing pad, and an
|
||||
/// appropriate action. The entry for an ordinary call has a try-range
|
||||
@ -490,37 +524,8 @@ ComputeCallSiteTable(SmallVectorImpl<CallSiteEntry> &CallSites,
|
||||
for (MachineBasicBlock::const_iterator MI = I->begin(), E = I->end();
|
||||
MI != E; ++MI) {
|
||||
if (!MI->isLabel()) {
|
||||
if (MI->getDesc().isCall()) {
|
||||
// Don't mark a call as potentially throwing if the function it's
|
||||
// calling is marked "nounwind".
|
||||
bool DoesNotThrow = false;
|
||||
bool SawFunc = false;
|
||||
for (unsigned OI = 0, OE = MI->getNumOperands(); OI != OE; ++OI) {
|
||||
const MachineOperand &MO = MI->getOperand(OI);
|
||||
|
||||
if (MO.isGlobal()) {
|
||||
if (Function *F = dyn_cast<Function>(MO.getGlobal())) {
|
||||
if (SawFunc) {
|
||||
// Be conservative. If we have more than one function operand
|
||||
// for this call, then we can't make the assumption that it's
|
||||
// the callee and not a parameter to the call.
|
||||
//
|
||||
// FIXME: Determine if there's a way to say that `F' is the
|
||||
// callee or parameter.
|
||||
DoesNotThrow = false;
|
||||
break;
|
||||
}
|
||||
if (F->doesNotThrow()) {
|
||||
SawFunc = true;
|
||||
DoesNotThrow = true;
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
|
||||
if (!DoesNotThrow)
|
||||
SawPotentiallyThrowing = true;
|
||||
}
|
||||
if (MI->getDesc().isCall())
|
||||
SawPotentiallyThrowing |= !CallToNoUnwindFunction(MI);
|
||||
|
||||
continue;
|
||||
}
|
||||
|
@ -155,6 +155,10 @@ class VISIBILITY_HIDDEN DwarfException : public Dwarf {
|
||||
SmallVectorImpl<ActionEntry> &Actions,
|
||||
SmallVectorImpl<unsigned> &FirstActions);
|
||||
|
||||
/// CallToNoUnwindFunction - Return `true' if this is a call to a function
|
||||
/// marked `nounwind'. Return `false' otherwise.
|
||||
bool CallToNoUnwindFunction(const MachineInstr *MI);
|
||||
|
||||
/// ComputeCallSiteTable - Compute the call-site table. The entry for an
|
||||
/// invoke has a try-range containing the call, a non-zero landing pad and an
|
||||
/// appropriate action. The entry for an ordinary call has a try-range
|
||||
|
Loading…
x
Reference in New Issue
Block a user