diff options
Diffstat (limited to 'llvm/lib/Target/X86')
| -rw-r--r-- | llvm/lib/Target/X86/X86MacroFusion.cpp | 13 |
1 files changed, 7 insertions, 6 deletions
diff --git a/llvm/lib/Target/X86/X86MacroFusion.cpp b/llvm/lib/Target/X86/X86MacroFusion.cpp index d09d8f36b77..ac44d920595 100644 --- a/llvm/lib/Target/X86/X86MacroFusion.cpp +++ b/llvm/lib/Target/X86/X86MacroFusion.cpp @@ -44,12 +44,13 @@ static bool shouldScheduleAdjacent(const X86Subtarget &ST, FuseInc } FuseKind; + assert((First || Second) && "At least one instr must be specified"); unsigned FirstOpcode = First - ? First->getOpcode() - : static_cast<unsigned>(X86::INSTRUCTION_LIST_END); - unsigned SecondOpcode = - Second ? Second->getOpcode() - : static_cast<unsigned>(X86::INSTRUCTION_LIST_END); + ? First->getOpcode() + : static_cast<unsigned>(X86::INSTRUCTION_LIST_END); + unsigned SecondOpcode = Second + ? Second->getOpcode() + : static_cast<unsigned>(X86::INSTRUCTION_LIST_END); switch (SecondOpcode) { default: @@ -215,7 +216,7 @@ void X86MacroFusion::apply(ScheduleDAGInstrs *DAGInstrs) { // For now, assume targets can only fuse with the branch. SUnit &ExitSU = DAG->ExitSU; MachineInstr *Branch = ExitSU.getInstr(); - if (!shouldScheduleAdjacent(ST, nullptr, Branch)) + if (!Branch || !shouldScheduleAdjacent(ST, nullptr, Branch)) return; for (SDep &PredDep : ExitSU.Preds) { |
