[PATCH] D17260: SystemZ scheduling implementation

Jonas Paulsson via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 18 05:50:13 PDT 2016


jonpa added inline comments.


================
Comment at: lib/Target/SystemZ/SystemZHazardRecognizer.h:120
+  /// means it would be good to schedule SU next.
+  void resourcesCost(SUnit *SU, int &Cost);
+
----------------
uweigand wrote:
> Why does groupingCost use the return value while resourcesCost uses an output parameter?
During experiments I have also returned a cost here for the 'other' processor side. I guess I could change that back now until it's needed again...?


https://reviews.llvm.org/D17260





More information about the llvm-commits mailing list