This is an archive of the discontinued LLVM Phabricator instance.

[mips] Update the generic model not to use instruction itineraries
Changes PlannedPublic

Authored by sdardis on Apr 6 2018, 4:48 AM.

Details

Summary

This model should now cover every single instruction.

Diff Detail

Event Timeline

sdardis created this revision.Apr 6 2018, 4:48 AM

This looks fine, regarding the instructions introduced instead of their matching itinerary classes.

There are messages in the form of "error:'MipsGenericModel' lacks information for '...'" though, as you mentioned in D45245.
The instructions reported seem to be entirely pseudo instructions.
Do those need to be included as well, perhaps marked with 'hasNoSchedulingInfo = 1' or is there an another way to deal with them?

sdardis planned changes to this revision.Apr 19 2018, 2:52 AM

Looks like I missed some instructions. I'll update this patch shortly.