[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