[PATCH] D35831: Don't use std::numeric_limits<double>::infinity() as result of function

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/CodeGen/TargetSchedule.cpp:369
        WPR != WEnd; ++WPR) {
-    unsigned Cycles = WPR->Cycles;
-    if (!Cycles)
-      return Optional<double>();
-
-    unsigned NumUnits =
-        SchedModel.getProcResource(WPR->ProcResourceIdx)->NumUnits;
-    Throughput = std::min(Throughput, NumUnits * 1.0 / Cycles);
+    if (unsigned Cycles = WPR->Cycles) {
+      unsigned NumUnits =
----------------
Remove Cycles and use WPR->Cycles directly (same for above).
```
if (WPR->Cycles)
```
and
```
double Temp = NumUnits * 1.0 / WPR->Cycles;
```



https://reviews.llvm.org/D35831