From 6d18d6e4d3d7cede100774a314d07c36b02ae4a4 Mon Sep 17 00:00:00 2001 From: Andrea Di Biagio Date: Sun, 11 Mar 2018 00:51:33 +0000 Subject: [TargetSchedule] Minor refactor in computeInstrLatency. NFC The intent of revision r300311 was to add a check for invalid scheduling class descriptors. However, it ended up adding a redundant call in a basic block that should not be reachable. llvm-svn: 327231 --- llvm/lib/CodeGen/TargetSchedule.cpp | 10 ++++------ 1 file changed, 4 insertions(+), 6 deletions(-) (limited to 'llvm/lib') diff --git a/llvm/lib/CodeGen/TargetSchedule.cpp b/llvm/lib/CodeGen/TargetSchedule.cpp index 86dbf1b2aea..7d636b126e0 100644 --- a/llvm/lib/CodeGen/TargetSchedule.cpp +++ b/llvm/lib/CodeGen/TargetSchedule.cpp @@ -274,14 +274,12 @@ unsigned TargetSchedModel::computeInstrLatency(unsigned Opcode) const { unsigned SCIdx = TII->get(Opcode).getSchedClass(); const MCSchedClassDesc *SCDesc = SchedModel.getSchedClassDesc(SCIdx); - if (SCDesc->isValid() && !SCDesc->isVariant()) + if (!SCDesc->isValid()) + return 0; + if (!SCDesc->isVariant()) return computeInstrLatency(*SCDesc); - if (SCDesc->isValid()) { - assert (!SCDesc->isVariant() && "No MI sched latency: SCDesc->isVariant()"); - return computeInstrLatency(*SCDesc); - } - return 0; + llvm_unreachable("No MI sched latency"); } unsigned -- cgit v1.2.3