[PATCH] D39235: [MachineScheduler] minor refactoring: new static function checkResourceLimit()

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 24 09:56:33 PDT 2017


MatzeB added a comment.

A clear "meh" from my side :)
Feel free to commit it.



================
Comment at: lib/CodeGen/MachineScheduler.cpp:1838
+                               unsigned Latency) {
+  return ((int)(Count - (Latency * LFactor)) > (int)LFactor);
+}
----------------
- Use doxygen `///` comments
- Unnecessary braces around return expression


https://reviews.llvm.org/D39235





More information about the llvm-commits mailing list