[all-commits] [llvm/llvm-project] aee340: [MISched][rework] Introduce and use ResourceSegments.
Francesco Petrogalli via All-commits
all-commits at lists.llvm.org
Fri Jun 9 06:02:50 PDT 2023
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: aee34000f9fbd50849af4e8950bf16d2034bae6a
https://github.com/llvm/llvm-project/commit/aee34000f9fbd50849af4e8950bf16d2034bae6a
Author: Francesco Petrogalli <francesco.petrogalli at apple.com>
Date: 2023-06-09 (Fri, 09 Jun 2023)
Changed paths:
M llvm/include/llvm/CodeGen/MachineScheduler.h
M llvm/include/llvm/CodeGen/TargetSchedule.h
M llvm/include/llvm/MC/MCSchedule.h
M llvm/include/llvm/Target/TargetSchedule.td
M llvm/lib/CodeGen/MachineScheduler.cpp
M llvm/lib/MC/MCSchedule.cpp
M llvm/unittests/CodeGen/CMakeLists.txt
A llvm/unittests/CodeGen/SchedBoundary.cpp
M llvm/utils/TableGen/SubtargetEmitter.cpp
Log Message:
-----------
[MISched][rework] Introduce and use ResourceSegments.
Re-landing the code that was reverted because of the buildbot failure
in https://lab.llvm.org/buildbot#builders/9/builds/27319.
Original commit message
======================
The class `ResourceSegments` is used to keep track of the intervals
that represent resource usage of a list of instructions that are
being scheduled by the machine scheduler.
The collection is made of intervals that are closed on the left and
open on the right (represented by the standard notation `[a, b)`).
These collections of intervals can be extended by `add`ing new
intervals accordingly while scheduling a basic block.
Unit tests are added to verify the possible configurations of
intervals, and the relative possibility of scheduling a new
instruction in these configurations. Specifically, the methods
`getFirstAvailableAtFromBottom` and `getFirstAvailableAtFromTop` are
tested to make sure that both bottom-up and top-down scheduling work
when tracking resource usage across the basic block with
`ResourceSegments`.
Note that the scheduler tracks resource usage with two methods:
1. counters (via `std::vector<unsigned> ReservedCycles;`);
2. intervals (via `std::map<unsigned, ResourceSegments> ReservedResourceSegments;`).
This patch can be considered a NFC test for existing scheduling models
because the tracking system that uses intervals is turned off by
default (field `bit EnableIntervals = false;` in the tablegen class
`SchedMachineModel`).
Reviewed By: andreadb
Differential Revision: https://reviews.llvm.org/D150312
More information about the All-commits
mailing list