[PATCH] D51160: Adjust MIScheduler to use ProcResource counts

Javed Absar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 29 01:17:45 PDT 2018


javed.absar added a comment.

Would it be possible to  add a test case that highlights the problem you are solving  e.g. take a processor that has multiple resources (e.g. two adders) and show that it now can use both resources in same cycle.



================
Comment at: lib/CodeGen/MachineScheduler.cpp:1945
+
+  for (unsigned Offset; Offset < NumberOfInstances; ++Offset) {
+    unsigned InstanceIndex = ResourceInstanceStartIndex + Offset;
----------------
why Offset is uninitialized ?


Repository:
  rL LLVM

https://reviews.llvm.org/D51160





More information about the llvm-commits mailing list