[PATCH] D35997: Support itineraries in TargetSubtargetInfo::getSchedInfoStr

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 28 09:28:28 PDT 2017


RKSimon added inline comments.


================
Comment at: lib/CodeGen/TargetSubtargetInfo.cpp:106
+  } else
+    Latency = TSchedModel.computeInstrLatency(MCI.getOpcode());
   Optional<double> RThroughput =
----------------
Cleanup the the nested ifs?
```
  if (TSchedModel.hasInstrSchedModel())
    Latency = TSchedModel.computeInstrLatency(MCI.getOpcode());
  else if (TSchedModel.hasInstrItineraries()) {
    auto *ItinData = TSchedModel.getInstrItineraries();
    Latency = ItinData->getStageLatency(
        getInstrInfo()->get(MCI.getOpcode()).getSchedClass());
  } else
      return std::string();
```




https://reviews.llvm.org/D35997





More information about the llvm-commits mailing list