[AIEX] Add support to select Itinerary based of RegClass of Operand #144
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
krishnamtibrewala
force-pushed
the
aie2-variable-itin
branch
5 times, most recently
from
5a6afe3 to
db77249
Compare
gbossu
reviewed
Aug 7, 2024
gbossu
reviewed
Aug 7, 2024
krishnamtibrewala
force-pushed
the
aie2-variable-itin
branch
2 times, most recently
from
85a2442 to
7a36434
Compare
krishnamtibrewala
force-pushed
the
aie2-variable-itin
branch
4 times, most recently
from
6702bd2 to
a2af5cc
Compare
gbossu
reviewed
Aug 19, 2024
gbossu
reviewed
Aug 19, 2024
krishnamtibrewala
force-pushed
the
aie2-variable-itin
branch
3 times, most recently
from
fbb7204 to
0eaa58a
Compare
gbossu
reviewed
Aug 20, 2024
gbossu
reviewed
Aug 20, 2024
gbossu
reviewed
Aug 20, 2024
krishnamtibrewala
force-pushed
the
aie2-variable-itin
branch
from
0eaa58a to
a962b24
Compare
krishnamtibrewala
force-pushed
the
aie2-variable-itin
branch
from
a962b24 to
c8b4387
Compare
krishnamtibrewala
requested review from
abhinay-anubola,
abnikant and
andcarminati
as code owners
krishnamtibrewala
requested review from
khallouh,
konstantinschwarz,
martien-de-jong,
SagarMaheshwari99 and
stephenneuendorffer
as code owners
gbossu
reviewed
Aug 21, 2024
…airs" with an Instr. ItineraryRegPairs consist of InstrItinClass, RegisterClass, OperandIdx, this is to allow a target to select Itinerary based on RegClass used for a given operand number
krishnamtibrewala
force-pushed
the
aie2-variable-itin
branch
from
c8b4387 to
2699ee6
Compare
gbossu
reviewed
Aug 21, 2024
gbossu
reviewed
Aug 21, 2024
gbossu
reviewed
Aug 21, 2024
gbossu
reviewed
Aug 21, 2024
krishnamtibrewala
force-pushed
the
aie2-variable-itin
branch
2 times, most recently
from
e0746d0 to
6167267
Compare
gbossu
reviewed
Aug 21, 2024
…sed on regClass This uses CodeGenInstruction data to generate: - InstrInfo::getSchedClass(const MCInstrDesc &Desc, iterator_range<const MachineOperand *> Operands)
-- InstrInfo::getSchedClass(const MCInstrDesc &Desc, iterator_range<const MachineOperand *> Operands) The function defaults to Desc.getSchedClass() in case instr is not defined with variable itinerary
krishnamtibrewala
force-pushed
the
aie2-variable-itin
branch
from
6167267 to
62115b0
Compare
gbossu
approved these changes
Aug 22, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
NFC, allowing to define instruction with variable itinerary based on regClass.
By default we assume operand 0 for an instruction but the newly added support allows to provide an operand no.