[PATCH] D33991: Improved throughput calculation

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon added inline comments.


================
Comment at: lib/CodeGen/TargetSchedule.cpp:355
+  if (Throughput == Unknown)
+    return Optional<double>();
   // We need reciprocal throughput that's why we return such value.
----------------
Aren't this change (and its equivalent in the next function) separate patches that need to go in first?


================
Comment at: lib/Target/X86/X86ScheduleBtVer2.td:345
+}
+def : InstRW<[WriteJVZEROUPPER], (instregex "VZEROUPPER")>;
+
----------------
This change to the btver2 model is a separate patch as well


https://reviews.llvm.org/D33991