This is an archive of the discontinued LLVM Phabricator instance.

[mips][sched] Split IIBranch into specific instruction classes.
ClosedPublic

Authored by dsanders on Aug 20 2015, 4:05 AM.

Details

Summary

Almost no functional change since the InstrItinData's have been duplicated.
The one functional change is to remove IIBranch from the MSA branches. The
classes will be assigned to the MSA instructions as part of implementing
the P5600 scheduler.

II_IndirectBranchPseudo and II_ReturnPseudo can probably be removed. I've
preserved the itinerary information for the corresponding pseudo
instructions to avoid making a functional change to these pseudos in
this patch.

Depends on D12188

Diff Detail

Event Timeline

dsanders updated this revision to Diff 32676.Aug 20 2015, 4:05 AM
dsanders retitled this revision from to [mips][sched] Split IIBranch into specific instruction classes..
dsanders updated this object.
dsanders added a reviewer: vkalintiris.
dsanders added a subscriber: llvm-commits.
vkalintiris accepted this revision.Aug 23 2015, 3:04 PM
vkalintiris edited edge metadata.

LGTM.

lib/Target/Mips/MicroMipsInstrInfo.td
167–169

We can use II_BCCZC for the time being as the default value for itin. We don't override this value anywhere as far as I can tell.

This revision is now accepted and ready to land.Aug 23 2015, 3:04 PM
dsanders added inline comments.Sep 22 2015, 5:44 AM
lib/Target/Mips/MicroMipsInstrInfo.td
167–169

It seems we don't use itin anywhere so I'll just remove the argument.

dsanders updated this revision to Diff 35369.Sep 22 2015, 5:58 AM
dsanders edited edge metadata.

Refresh before commit.

dsanders closed this revision.Sep 22 2015, 6:37 AM