This is an archive of the discontinued LLVM Phabricator instance.

[mips] Sink PredicateControl further down the class hierarchy.
ClosedPublic

Authored by sdardis on May 30 2018, 2:55 AM.

Details

Summary

Previously PredicateControl in some cases was a member of <X>Inst classes
for some X (DSP, EVA) or was in more irregular place in the hierarchry
for any given instruction.

This patch moves PredicateControl down to the root so that it is consistently
available. Then correct the base class of microMIPS instructions as using
EncodingPredicates instead of the general Predicates field of Instruction.

Diff Detail

Repository
rL LLVM

Event Timeline

sdardis created this revision.May 30 2018, 2:55 AM

Looks okay except it seems that other changes have snuck in.
From what I can tell the patch was created before rL333288 got committed.
In addition this patch appears to contain changes from D47075 as well.
Please rebase.

sdardis updated this revision to Diff 149088.May 30 2018, 4:49 AM

Rebase after updating trunk.

This revision is now accepted and ready to land.May 30 2018, 5:09 AM
This revision was automatically updated to reflect the committed changes.